build(build): add Pester code coverage to main branch CI#250
Merged
WilliamBerryiii merged 1 commit intomainfrom Jan 23, 2026
Merged
build(build): add Pester code coverage to main branch CI#250WilliamBerryiii merged 1 commit intomainfrom
WilliamBerryiii merged 1 commit intomainfrom
Conversation
Contributor
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds Pester code coverage generation to main branch CI pushes, establishing a baseline for Codecov to compare against in pull requests. It introduces a codecov.yml configuration file with appropriate thresholds and flags, adds the pester-tests job to the main branch workflow, and includes a necessary spelling dictionary entry.
Changes:
- Added
pester-testsjob tomain.ymlworkflow with code coverage enabled, running all tests (not just changed files) on main branch pushes - Created
codecov.ymlconfiguration with auto-targeting project status, informational patch status, carryforward flags for pester, and appropriate ignore paths - Added "rhysd" to
.cspell.jsondictionary to support the actionlint tool reference inyaml-lint.yml
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
.github/workflows/main.yml |
Adds pester-tests job to main branch CI with OIDC permissions and code coverage enabled; updates extension-package job dependencies to include pester tests |
codecov.yml |
New Codecov configuration defining coverage status thresholds, patch targets, ignore paths, comment behavior, and pester flag with carryforward |
.cspell.json |
Adds "rhysd" to the words dictionary for actionlint repository owner reference |
- add pester-tests job to main.yml with OIDC permissions - add codecov.yml with status thresholds and carryforward flags - add rhysd to cspell dictionary for actionlint checksums Fixes #249 📊 - Generated by Copilot
135151c to
2b2cb64
Compare
katriendg
approved these changes
Jan 23, 2026
This was referenced Jan 24, 2026
WilliamBerryiii
added a commit
that referenced
this pull request
Jan 25, 2026
## Description Adds workflow-level concurrency controls to all three orchestrator workflows. This prevents unnecessary parallel runs when multiple commits are pushed in quick succession and reduces GitHub Actions resource consumption. **Changes:** - Added `concurrency` block to `pr-validation.yml` with `cancel-in-progress: true` to cancel stale PR validation runs when new commits arrive - Added `concurrency` block to `main.yml` with `cancel-in-progress: false` to prevent cancellation of main branch builds - Added `concurrency` block to `weekly-security-maintenance.yml` with `cancel-in-progress: false` for scheduled job protection - Updated `.cspell/general-technical.txt` with missing dictionary terms (`carryforward`, `SCHED`, `SECCHECK`) that were causing pre-existing spell-check failures ## Related Issue(s) Fixes #274 ## Type of Change Select all that apply: **Code & Documentation:** - [ ] Bug fix (non-breaking change fixing an issue) - [ ] New feature (non-breaking change adding functionality) - [ ] Breaking change (fix or feature causing existing functionality to change) - [ ] Documentation update **Infrastructure & Configuration:** - [x] GitHub Actions workflow - [x] Linting configuration (markdown, PowerShell, etc.) - [ ] Security configuration - [ ] DevContainer configuration - [ ] Dependency update **AI Artifacts:** - [ ] Reviewed contribution with `prompt-builder` agent and addressed all feedback - [ ] Copilot instructions (`.github/instructions/*.instructions.md`) - [ ] Copilot prompt (`.github/prompts/*.prompt.md`) - [ ] Copilot agent (`.github/agents/*.agent.md`) > **Note for AI Artifact Contributors**: > > - **Agents**: Research, indexing/referencing other project (using standard VS Code GitHub Copilot/MCP tools), planning, and general implementation agents likely already exist. Review `.github/agents/` before creating new ones. > - **Model Versions**: Only contributions targeting the **latest Anthropic and OpenAI models** will be accepted. Older model versions (e.g., GPT-3.5, Claude 3) will be rejected. > - See [Agents Not Accepted](../docs/contributing/custom-agents.md#agents-not-accepted) and [Model Version Requirements](../docs/contributing/ai-artifacts-common.md#model-version-requirements). **Other:** - [ ] Script/automation (`.ps1`, `.sh`, `.py`) - [ ] Other (please describe): ## Sample Prompts (for AI Artifact Contributions) N/A - This PR does not include AI artifacts. ## Testing - [x] YAML lint validation passes (`npm run lint:yaml`) - [x] Spell check passes (`npm run spell-check`) ## Checklist ### Required Checks - [ ] Documentation is updated (if applicable) - [x] Files follow existing naming conventions - [x] Changes are backwards compatible (if applicable) - [ ] Tests added for new functionality (if applicable) ### AI Artifact Contributions N/A ### Required Automated Checks The following validation commands must pass before merging: - [x] Markdown linting: `npm run lint:md` - [x] Spell checking: `npm run spell-check` - [x] Frontmatter validation: `npm run lint:frontmatter` - [ ] Link validation: `npm run lint:md-links` - [ ] PowerShell analysis: `npm run lint:ps` ## Security Considerations - [x] This PR does not contain any sensitive or NDA information - [ ] Any new dependencies have been reviewed for security issues - [x] Security-related scripts follow the principle of least privilege ## Additional Notes ### Concurrency Configuration Patterns | Workflow | Group Key | Cancel In-Progress | Rationale | |----------|-----------|-------------------|-----------| | `pr-validation.yml` | `workflow-head_ref` | `true` | Cancel stale PR runs when new commits arrive | | `main.yml` | `workflow-ref` | `false` | Protect main branch builds from cancellation | | `weekly-security-maintenance.yml` | `workflow` | `false` | Single weekly job, no ref variation needed | ### Dictionary Updates Fixed pre-existing spell-check gaps that passed CI due to timing: - `carryforward` - Codecov configuration term (PR #250) - `SCHED`, `SECCHECK` - Mermaid diagram labels (PR #252) ⚡ - Generated by Copilot
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
Adds Pester code coverage generation to main branch pushes, establishing baseline coverage for Codecov PR comparisons. Also introduces a
codecov.ymlconfiguration with status thresholds, carryforward flags, and ignore paths.Changes:
pester-testsjob tomain.ymlwith OIDC permissions (id-token: write) andcode-coverage: truecodecov.ymlwith project status (target: auto, 1% threshold), informational patch status (80% target), carryforward flags for pester, and ignore paths for test code and logs.cspell.jsondictionary for actionlint checksum referencesRelated Issue(s)
Fixes #249
Type of Change
Select all that apply:
Code & Documentation:
Infrastructure & Configuration:
AI Artifacts:
prompt-builderagent and addressed all feedback.github/instructions/*.instructions.md).github/prompts/*.prompt.md).github/agents/*.agent.md)Other:
.ps1,.sh,.py)Sample Prompts (for AI Artifact Contributions)
N/A - This PR does not include AI artifacts.
Testing
pwsh -File scripts/linting/Invoke-YamlLint.ps1)npm run spell-check)npm run lint:frontmatter)npm run lint:md)Checklist
Required Checks
AI Artifact Contributions
N/A
Required Automated Checks
The following validation commands must pass before merging:
npm run lint:mdnpm run spell-checknpm run lint:frontmatternpm run lint:md-linksnpm run lint:psSecurity Considerations
Additional Notes
The
codecov.ymlconfiguration was added based on research of Codecov best practices:coverage.status.project.targetautocoverage.status.project.threshold1%coverage.status.patch.informationaltrueflags.pester.carryforwardtrueignorescripts/tests/**,logs/**📊 - Generated by Copilot