Skip to content

Ignore test coverage output and drop committed report#382

Merged
ptr727 merged 1 commit into
developfrom
fix/ignore-coverage-output
Jul 9, 2026
Merged

Ignore test coverage output and drop committed report#382
ptr727 merged 1 commit into
developfrom
fix/ignore-coverage-output

Conversation

@ptr727

@ptr727 ptr727 commented Jul 9, 2026

Copy link
Copy Markdown
Owner

A local coverage run (dotnet test --results-directory ./coverage) was accidentally swept into #381 via git add -A because ./coverage was not gitignored. Remove the committed coverage/**/coverage.cobertura.xml and add [Cc]overage/ + TestResults/ to .gitignore so coverage artifacts stay out of the tree.

No shipped inputs change, so this does not trigger a release.

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings July 9, 2026 23:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@codecov

codecov Bot commented Jul 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 40.23%. Comparing base (d941159) to head (f31e70c).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #382      +/-   ##
===========================================
- Coverage    42.77%   40.23%   -2.54%     
===========================================
  Files           10       10              
  Lines         1010     1009       -1     
  Branches       100       97       -3     
===========================================
- Hits           432      406      -26     
- Misses         532      577      +45     
+ Partials        46       26      -20     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ptr727
ptr727 merged commit 9480c4b into develop Jul 9, 2026
11 checks passed
ptr727 added a commit that referenced this pull request Jul 10, 2026
Promotes the 3.7 release from develop to main.

**Contents**
- #381 - Adopt an injectable `Microsoft.Extensions.Logging` logging
model: `LogOptions.SetFactory(ILoggerFactory)` replaces the removed
Serilog-typed `LogOptions.Logger`; source-generated `[LoggerMessage]`
logging in `FileEx`/`Download`; `Extensions` renamed to
`CompressExtensions`; Serilog-wired `Sandbox`; test suite migrated to
AwesomeAssertions.
- #382 - Gitignore test coverage output and drop an
accidentally-committed report.

**Release impact:** merging publishes the stable **3.7** package to
NuGet (breaking API changes: `LogOptions.Logger` removed, `Extensions`
renamed). No issues to close.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
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