Skip to content

feat[ci]: use coverage combine to reduce codecov uploads#4452

Merged
charles-cooper merged 62 commits intovyperlang:masterfrom
tserg:ci/coverage
Jan 20, 2025
Merged

feat[ci]: use coverage combine to reduce codecov uploads#4452
charles-cooper merged 62 commits intovyperlang:masterfrom
tserg:ci/coverage

Conversation

@tserg
Copy link
Copy Markdown
Contributor

@tserg tserg commented Jan 17, 2025

What I did

Consolidate code coverage files and upload it once.

How I did it

How to verify it

Commit message

This commit updates the CI to combine code coverage to a single file
and perform just a single upload after all tests finish (rather than
each test run uploading its own coverage report). This should reduce
failures / rate limiting on the codecov app, and also prevent the
codecov app from producing an inaccurate coverage report before all
the tests finish.

references:
- https://coverage.readthedocs.io/en/7.6.10/cmd.html#cmd-combine
- https://coverage.readthedocs.io/en/7.6.10/cmd.html#re-mapping-paths
- https://coverage.readthedocs.io/en/7.6.10/config.html#config-run-relative-files

Description for the changelog

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@charles-cooper charles-cooper changed the title chore[ci]: upload code coverage once feat[ci]: use coverage combine to reduce codecov uploads Jan 20, 2025
@charles-cooper charles-cooper enabled auto-merge (squash) January 20, 2025 15:25
@charles-cooper charles-cooper enabled auto-merge (squash) January 20, 2025 15:34
@charles-cooper charles-cooper merged commit d7f50df into vyperlang:master Jan 20, 2025
@tserg tserg deleted the ci/coverage branch January 21, 2025 03:22
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