Skip to content

Add Coveralls coverage and badge - #43

Merged
dblock merged 1 commit into
ruby-grape:masterfrom
dblock:ci/coveralls-coverage
Oct 27, 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

  • Generate SimpleCov LCOV reports before application and optional ORM code loads; ignore local coverage artifacts.
  • Keep core, ActiveRecord, and Mongoid matrices in separate files. test.yml calls test-activerecord.yml and test-mongodb.yml as reusable workflows.
  • Only test.yml handles push/pull-request events, avoiding duplicate workflow runs. All 22 compatibility matrix entries and their test steps are unchanged.
  • Upload from existing ORM entries marked coverage: true: ActiveRecord 7.2, Mongoid 6, and Mongoid 9. This covers both Mongoid association implementations without extra test runs.
  • Finish one merged Coveralls build in test.yml after the core matrix and both reusable workflows succeed.
  • Add a README Coveralls badge and document orchestration. One test badge covers core and both ORM workflows.

Uses the built-in GitHub token with no Coveralls secret.

Validation

Verified the reusable workflow job definitions exactly match the previous combined jobs, preserve all 22 matrix entries, expose only workflow_call, and are all required by the coverage finalizer.

CI: Separate reusable ORM workflows and merged Coveralls. All 22 test jobs and the single completion job passed. Verified merged coverage remains 97.31% (289/297 lines).

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 3 times, most recently from a970af8 to be65090 Compare October 26, 2026 16:35
Start SimpleCov before application and ORM loading and write LCOV reports. Merge ActiveRecord and both Mongoid association generations in a dedicated coverage workflow without changing existing compatibility matrices.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dblock
dblock force-pushed the ci/coveralls-coverage branch from be65090 to 31b320d Compare October 26, 2026 22:09
@dblock
dblock merged commit ffe5358 into ruby-grape:master Oct 27, 2026
27 checks passed
@dblock
dblock deleted the ci/coveralls-coverage branch October 27, 2026 02:12
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