Skip to content

Report coverage to Coveralls on pull requests using GITHUB_TOKEN - #65

Merged
dblock merged 1 commit into
masterfrom
dblock/coveralls-github-token
Aug 25, 2026
Merged

Report coverage to Coveralls on pull requests using GITHUB_TOKEN#65
dblock merged 1 commit into
masterfrom
dblock/coveralls-github-token

Conversation

@dblock

@dblock dblock commented Aug 25, 2026

Copy link
Copy Markdown
Owner

Description

The Coverage workflow only triggers on: [push] and uses coveralls_reborn with a COVERALLS_REPO_TOKEN secret. This means it never runs on pull requests, and even if it did, PRs from forks don't have access to that secret, so coverage is never reported to Coveralls or shown as a PR check.

  • Switch from coveralls_reborn to the official coverallsapp/github-action, authenticated with the default GITHUB_TOKEN (available on every workflow run, including from fork PRs — no extra secret to configure).
  • Switch the SimpleCov formatter from coveralls_reborn's to simplecov-lcov, since coverallsapp/github-action consumes an lcov report.
  • Add pull_request as a trigger so coverage now runs and reports on PRs, not only on pushes to master.

Verified locally: 70 examples, 0 failures, 100% line coverage (unchanged).

@github-actions

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown

Danger Report

No issues found.

View run

Switches from coveralls_reborn (which requires a Coveralls-specific
COVERALLS_REPO_TOKEN secret that isn't available to workflow runs
triggered by pull requests from forks) to the coverallsapp/github-action,
which authenticates with the default GITHUB_TOKEN available on any PR.

- Replace coveralls_reborn with simplecov-lcov to generate an lcov
  report that coverallsapp/github-action can consume.
- Add pull_request as a trigger for the Coverage workflow so coverage
  is reported on PRs, not just on pushes.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dblock
dblock force-pushed the dblock/coveralls-github-token branch from de7b1d6 to aabc2eb Compare August 25, 2026 22:27
@dblock
dblock merged commit e28eee3 into master Aug 25, 2026
21 checks passed
@dblock
dblock deleted the dblock/coveralls-github-token branch August 25, 2026 22:30
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.

1 participant