Skip to content

Add code coverage script.#1369

Closed
jzmaddock wants to merge 1 commit intodevelopfrom
new-code-coverage
Closed

Add code coverage script.#1369
jzmaddock wants to merge 1 commit intodevelopfrom
new-code-coverage

Conversation

@jzmaddock
Copy link
Collaborator

No description provided.

@mborland
Copy link
Member

Thanks for adding this John! One thing we will have to watch for is timeout since these coverage runs can be lengthy. I had to break the Codecov up by test-suite to avoid that. If that does happen we can always ping Sam and ask him if there's a way to do the same thing (or if he can implement it) with this coverage generator.

@codecov
Copy link

codecov bot commented Feb 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.34%. Comparing base (a10f3c8) to head (20e9efd).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1369      +/-   ##
===========================================
- Coverage    95.34%   95.34%   -0.01%     
===========================================
  Files          825      825              
  Lines        68160    68160              
===========================================
- Hits         64987    64986       -1     
- Misses        3173     3174       +1     

see 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a10f3c8...20e9efd. Read the comment docs.

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

@jzmaddock
Copy link
Collaborator Author

Not so fast, nothing was run! Any ideas @mborland ?

@mborland
Copy link
Member

Not so fast, nothing was run! Any ideas @mborland ?

It was set to only run on push to develop or master. Now I have it set to the same conditions that run our Codecov runs. I wasn't sure if that was a limitation of the new coverage machinery or not.

@mborland
Copy link
Member

It does look like we are limited to pushes to develop and master for right now. I will remove my commit

@jzmaddock
Copy link
Collaborator Author

Oh :(

@jzmaddock jzmaddock closed this Feb 25, 2026
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