Skip to content

ci: fix the coverage job (PowerShell pin and stale net10.0 path) - #200

Merged
davidnmbond merged 2 commits into
mainfrom
fix/pin-powershell-tool
Nov 13, 2026
Merged

davidnmbond merged 2 commits into
mainfrom
fix/pin-powershell-tool

Conversation

@davidnmbond

@davidnmbond davidnmbond commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

The coverage job has been failing for two separate reasons, found one behind the other.

  1. PowerShell install. The job installs the latest PowerShell dotnet tool. 7.7.0 is published as a tool but its linux-x64 runtime package is not on nuget.org yet, so the install fails (run 43415947603). Pinned to 7.6.10, the last version that installs. Unpin once 7.7.x installs.
  2. Stale framework path. With the install fixed, Run-Coverage.ps1 failed because it hardcoded bin/Debug/net10.0/ for the test host, and the repository targets net11.0 (run 43416727454). It now picks the newest netX.Y folder that holds the host. The rule that catches this class of problem in workflows (CI-17) does not scan scripts; that is a gap worth closing separately.

CI on this pull request is the proof.

7.7.0 is published as a dotnet tool but its linux-x64 runtime package is not on nuget.org, so the unpinned install fails the coverage job before any test runs.
@codacy-production

codacy-production Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Coverage ∅ diff coverage

Metric Results
Coverage variation Report missing for cb8dbc61
Diff coverage ✅ ∅ diff coverage

View coverage diff in Codacy

Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (cb8dbc6) Report Missing Report Missing Report Missing
Head commit (55007ad) 22624 14382 63.57%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#200) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

1 Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct.

AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

@codacy-production codacy-production Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

✅ No blocking issues noticed

No blocking findings were identified. Workflow behavior should still be validated to confirm the pinned tool is installed and usable before coverage execution.

Test suggestions

  • Verify the coverage workflow installs the pinned PowerShell version successfully.
  • Verify the coverage job can invoke pwsh after installation and run the coverage tests.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify the coverage workflow installs the pinned PowerShell version successfully.
2. Verify the coverage job can invoke `pwsh` after installation and run the coverage tests.

TIP Add a Jira or Linear ticket key to the branch or title so the reviewer can check the PR against it. Using another tracker? Tell us.
TIP How was this review? Give us feedback

Run-Coverage.ps1 named bin/Debug/net10.0, so the coverage job failed once the repository moved to net11.0 (the host was never found). It now picks the newest framework folder that holds the host.
@davidnmbond davidnmbond changed the title ci: pin the PowerShell tool to 7.6.10 ci: fix the coverage job (PowerShell pin and stale net10.0 path) Nov 13, 2026
@davidnmbond
davidnmbond merged commit 15455f7 into main Nov 13, 2026
7 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