Skip to content

Conversation

@MariusStorhaug
Copy link
Member

@MariusStorhaug MariusStorhaug commented Oct 7, 2025

Description

This pull request makes a minor improvement to error handling in the PowerShell documentation build script. Instead of writing errors as errors, it now logs them as warnings with more context when markdown help generation fails for a command.

  • Improved error reporting by changing Write-Error to Write-Warning with a descriptive message in Build-PSModuleDocumentation.ps1.

Copilot AI review requested due to automatic review settings October 7, 2025 09:11
Copy link
Contributor

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.

Pull Request Overview

This PR removes all source files from a test repository directory and fixes an error handling issue in the documentation build script. The changes appear to be part of cleaning up test artifacts while improving error reporting.

  • Deletion of all PowerShell source files, XML configuration files, and documentation from the test repository
  • Removal of automatically generated module content that contained redundant documentation
  • Improvement of error handling in the documentation build script by changing error output to warning

Reviewed Changes

Copilot reviewed 27 out of 28 changed files in this pull request and generated no comments.

File Description
tests/srcTestRepo/src/* Complete removal of all source files including functions, classes, variables, types, and configuration
tests/srcTestRepo/outputs/* Removal of generated module files and redundant documentation blocks
scripts/helpers/Build-PSModuleDocumentation.ps1 Changed error handling from Write-Error to Write-Warning for better build process flow

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copilot AI review requested due to automatic review settings October 7, 2025 10:51
Copy link
Contributor

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.

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@MariusStorhaug MariusStorhaug changed the title Fix error 🩹[Patch]: Improve error handling and output error summary Oct 7, 2025
@MariusStorhaug MariusStorhaug self-assigned this Oct 7, 2025
@MariusStorhaug MariusStorhaug enabled auto-merge (squash) October 7, 2025 10:58
@MariusStorhaug MariusStorhaug merged commit aa32aa1 into main Oct 7, 2025
18 checks passed
@MariusStorhaug MariusStorhaug deleted the fixError branch October 7, 2025 11:00
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