← Back to issue list

ci: resurrect codecov

View original Github issue

Metadata

Project
snapcraft
Number
#2138
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #2138 (commit 57f439a4) reinstated Codecov coverage reporting in CI after PR #939's switch from Coveralls lapsed, gating publishing on coverage checks. 1 file changed, +3/-1.

Resolution: closed resolved

Reason: The pull request itself was merged by maintainer sergiusens as commit 57f439a4 ("ci: resurrect codecov (#2138)"), restoring the Codecov upload/report step so coverage is collected and checked before publishing.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Bring back codecov and ensure the proper checks are done before publishing. Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [ ] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #2138 (commit 57f439a4) reinstated Codecov coverage reporting in CI after PR #939's switch from Coveralls lapsed, gating publishing on coverage checks. 1 file changed, +3/-1.
stealth/space-bunny-alpha — closed resolved Merged PR #2138 (57f439a4) restored Codecov coverage reporting and required coverage checks before publishing.
qwen/qwen3.6-35b-a3b — — Merged to restore Codecov integration in the CI pipeline. The three-line update re-enables coverage reporting and ensures checks run before publishing, successfully resolving the missing coverage data issue.
qwen3.6-35b-a3b-mtp-q6 — — Merged to restore Codecov integration in the CI pipeline. The update re-enables coverage reporting and enforces validation checks before publishing.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#939 (confidence 78%)

    Earlier commit e3a71c5e4 (PR #939) replaced Coveralls with Codecov; coverage uploads had silently stopped by 2018, which this PR fixes.

Related issues

Issue Project State Summary Similarity
#939 Replace coveralls with codecov snapcraft merged Merged PR #939 replaced Coveralls with Codecov for test coverage reporting and updated the coverage badge; the change landed as commit e3a71c5e4.
77%
#307 build(deps): update codecov/codecov-action action to v5 craft-cli merged Merged PR #307 upgraded codecov/codecov-action from v4 to v5 in CI via merge commit 9ecd49d.
70%