Skip to content

Merge stable matrix coverage in Coveralls - #56

Merged
dblock merged 1 commit into
ruby-grape:masterfrom
dblock:ci/coveralls-coverage
Oct 26, 2026
Merged

dblock merged 1 commit into
ruby-grape:masterfrom
dblock:ci/coveralls-coverage

Conversation

@dblock

@dblock dblock commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

Changes

  • Extend existing LCOV reporting from one job to every stable Ruby/Grape matrix entry, with unique flags and a final Coveralls completion job.
  • Keep ruby-head experimental and exclude it from uploads.
  • Preserve existing SimpleCov accounting rather than counting the version file loaded by Bundler before instrumentation.
  • Refresh the README Coveralls badge and document generated reports.

Uses the built-in GitHub token; no Coveralls secret or additional dependencies are needed.

Validation

34 examples and RuboCop pass locally. Verified LCOV generation, all 99 instrumented lines covered, and valid workflow YAML.

CI: Tests and merged Coveralls.

Created by Copilot, an AI, on behalf of @dblock.

@github-actions

github-actions Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Danger Report

No issues found.

View run

@dblock
dblock force-pushed the ci/coveralls-coverage branch from 9c59d11 to 487cfde Compare October 26, 2026 16:15
@coveralls

coveralls commented Oct 26, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 100.0%. remained the same — dblock:ci/coveralls-coverage into ruby-grape:master

Publish and finalize parallel LCOV reports from all stable Ruby/Grape jobs. Preserve existing SimpleCov file accounting and refresh the Coveralls badge and contributor guidance.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dblock
dblock force-pushed the ci/coveralls-coverage branch from 487cfde to 8dd4ce8 Compare October 26, 2026 16:21
@dblock
dblock merged commit e09be42 into ruby-grape:master Oct 26, 2026
22 checks passed
@dblock
dblock deleted the ci/coveralls-coverage branch October 26, 2026 16:31
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