Skip to content

Conversation

@MariusStorhaug
Copy link
Member

@MariusStorhaug MariusStorhaug commented Apr 24, 2025

Description

This pull request introduces several changes to improve logging, enhance test configuration, and streamline code readability in the PowerShell module and its associated tests. Key updates include adding code coverage settings.

Logging Improvements

  • Replaced all occurrences of Write-Verbose with Write-Debug in tests/outputTestRepo/outputs/module/PSModuleTest/PSModuleTest.psm1.

Test Configuration Enhancements

  • ⚠️ Added a CodeCoverage section with Enabled = $true in scripts/tests/Module/Module.Configuration.ps1 to enable code coverage reporting during tests.

Type of change

  • 📖 [Docs]
  • 🪲 [Fix]
  • 🩹 [Patch]
  • ⚠️ [Security fix]
  • 🚀 [Feature]
  • 🌟 [Breaking change]

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

@MariusStorhaug MariusStorhaug self-assigned this Apr 24, 2025
Copilot AI review requested due to automatic review settings April 24, 2025 06:37
@MariusStorhaug MariusStorhaug requested a review from a team as a code owner April 24, 2025 06:37
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • scripts/tests/Module/PSModule/PSModule.Tests.ps1: Language not supported

…e handling in test scripts for improved clarity
…nd adjust debug/verbose handling in PSModule.Tests.ps1 for cleaner execution
…er.ps1 and streamline module import in PSModule.Tests.ps1
@MariusStorhaug MariusStorhaug merged commit c207858 into main Apr 24, 2025
21 checks passed
@MariusStorhaug MariusStorhaug deleted the debugImport branch April 24, 2025 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants