Skip to content

Comments

Merge develop into main: CI fixes and local-first testing#6

Merged
christaylorcodes merged 6 commits intomainfrom
develop
Feb 4, 2026
Merged

Merge develop into main: CI fixes and local-first testing#6
christaylorcodes merged 6 commits intomainfrom
develop

Conversation

@christaylorcodes
Copy link
Owner

  • Fix PSScriptAnalyzer warnings for manifest wildcards and missing BOM
  • Fix PSScriptAnalyzer warnings for manifest wildcards and missing BOM
  • Consolidate CI pipeline and enforce local-first testing
  • Fix release job exit code leak from gh release view

Pull Request

Description

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Refactoring / tests

Testing

  • ./Tests/test-local.ps1 passes (build + analyze + test)

Checklist

  • Code follows project conventions (AGENTS.md)
  • Tests cover new/changed behavior
  • PSScriptAnalyzer reports zero errors
  • Documentation updated (if applicable)
  • Module built (./build.ps1 -Tasks build)

AI Contribution

  • AI-assisted (tool: , issue: #)

christaylorcodes and others added 6 commits February 3, 2026 16:03
Replace wildcard exports in source manifest with explicit function and
alias lists to satisfy PSUseToExportFieldsInManifest. Add UTF-8 BOM to
three files containing non-ASCII characters (em-dashes in comments) to
resolve PSUseBOMForUnicodeEncodedFile warnings.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Replace wildcard exports in source manifest with explicit function and
alias lists to satisfy PSUseToExportFieldsInManifest. Add UTF-8 BOM to
three files containing non-ASCII characters (em-dashes in comments) to
resolve PSUseBOMForUnicodeEncodedFile warnings.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Resolve manifest merge conflict: single-line explicit exports
- Consolidate CI test+analyze into single smoke-test job
- Fix test-local.ps1 severity mismatch: fail on warnings like CI
- Add mandatory validation section to CLAUDE.md
- Strengthen AGENTS.md workflow with explicit validation gates
- Expand copilot-instructions.md with structured test command

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
When gh release view returns exit code 1 (release not found), PowerShell
propagates that as the step exit code. Add explicit exit 0 since the
skip/proceed logic is handled via step outputs, not exit codes.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
PSScriptAnalyzer intermittently throws NullReferenceException when
parsing .psd1 manifest files in CI. Analyze only .ps1/.psm1 files
since ModuleBuilder overwrites the manifest at build time anyway.

Updated in all three analyzer locations: CI smoke-test, test-local.ps1,
and Invoke-QuickTest.ps1.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@christaylorcodes christaylorcodes merged commit d879680 into main Feb 4, 2026
13 of 14 checks passed
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.

1 participant