Conversation
…772493305728 Add Claude Code GitHub Workflow
- Update link to code styles
Introduces a new `CLAUDE.md` file to provide specific guidance for Claude Code when interacting with this repository. Details include: - A high-level overview of the `AsBuiltReport.Core` module. - Instructions for running tests, code quality checks, and installing dependencies. - Comprehensive architectural insights into module loading, public/private functions, and localization. - Explanations of the test suite structure, coding standards, and branch/PR workflow. This documentation aims to improve AI's understanding and interaction with the codebase.
…d New-AsBuiltReportConfig scripts - Updated string comparisons from `""` to `''` for consistency and clarity. - Enhanced validation logic for user inputs in various prompts to ensure default values are set correctly. - Excluded 'AsBuiltReport.Chart' and 'AsBuiltReport.Diagram' from the list of installed report modules in multiple locations to streamline report generation.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Refactor input validation in New-AsBuiltConfig, New-AsBuiltReport, and New-AsBuiltReportConfig scripts
Proposed solution to #74
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
PSScriptAnalyzer results: Errors: [0], Warnings: [202], Information: [1]
|
rebelinux
suggested changes
May 14, 2026
Contributor
rebelinux
left a comment
There was a problem hiding this comment.
The module version needs to be updated to 1.6.3.
rebelinux
approved these changes
May 14, 2026
- Update module version in manifest
|
PSScriptAnalyzer results: Errors: [0], Warnings: [202], Information: [1]
|
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Maintenance release with input validation refactoring, function naming fixes, and removal of unused GitHub Actions workflows.
Related Issue
Fixes #74
Motivation and Context
New-AsBuiltConfig,New-AsBuiltReport, andNew-AsBuiltReportConfigwas inconsistent in string comparisons and default value handling; refactored for consistencyAsBuiltReport.ChartandAsBuiltReport.Diagramwere incorrectly appearing in installed report module lists; now excludedDraw-AsciiBoxwas being called in the email credentials prompt but the function was renamed toDraw-AsciiBannerin v1.5.0; corrected$PSBoundParameters.ContainsKey('Verbose')for reliable detection across all invocation stylesHow Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: