← Back to issue list

travis: remove the coveralls token

View original Github issue

Metadata

Project
snapcraft
Number
#309
Type
pull request
State
merged
Author
come-maiz
Labels
—
Created
Updated
Closed

Current evaluation

PR merged into master as commit f11c05e8a (merge commit 1b70425bd), dropping the Coveralls token from .travis.yml since the service no longer requires it for PR builds.

Resolution: closed resolved

Reason: The PR was merged by maintainer come-maiz after kyrofa approved it, landing as commit f11c05e8a with merge commit 1b70425bd "Merge pull request #309 from elopio/no-coveralls-token". The change removed the token line from .travis.yml (+2/-5), resolving Launchpad bug LP: #1543675.

Evaluated by: mr-cal-bot

No scores available.

Issue body

LP: #1543675

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved PR merged into master as commit f11c05e8a (merge commit 1b70425bd), dropping the Coveralls token from .travis.yml since the service no longer requires it for PR builds.
stealth/space-bunny-alpha — closed resolved Merged in #309 via commit f11c05e8a, removing the Coveralls token from Travis CI after maintainers confirmed it was unnecessary.
qwen/qwen3.6-35b-a3b — — Merged removal of the Coveralls token from Travis CI configuration. Discussion confirmed the token is unnecessary as Coveralls safely isolates master branch coverage and does not require authentication for Travis triggers.
qwen3.6-35b-a3b-mtp-q6 — — Merged removal of the Coveralls token from Travis CI. The token was unnecessary because Coveralls isolates master coverage from pull requests and the Travis trigger API requires no authentication. Approved after reviewer clarification.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1543675 The coveralls token is not needed for the travis executions snapcraft (launchpad) closed Coveralls token removed from .travis.yml by ubuntu-core/snapcraft PR #309, merged as commit f11c05e8a (merge 1b70425bd). Launchpad bug #1543675 closed by elopio with a link to that PR.
81%
#59 Adding travis and coverall badges snapcraft merged Merged into main, adding Travis CI and Coveralls badges to the README in merge commit 7b13a45c6.
73%