Skip to content

Modernize CI and fix Tests badge link in README - #48

Merged
dblock merged 2 commits into
masterfrom
dblock/modernize-ci
Aug 21, 2026
Merged

Modernize CI and fix Tests badge link in README#48
dblock merged 2 commits into
masterfrom
dblock/modernize-ci

Conversation

@dblock

@dblock dblock commented Aug 21, 2026

Copy link
Copy Markdown
Owner

What

  • Bump actions/checkout from v3 to v7 in test.yml, rubocop.yml, and danger.yml.
  • Update the test matrix: drop EOL Ruby 2.7/3.0/3.1, add Ruby 3.3/3.4 (keeping 3.1 as the floor, since the gemspec doesn't pin a minimum), and keep ruby-head/jruby-head as allowed failures.
  • Bump the pinned Ruby version used by the RuboCop and Danger jobs from 3.1 to 3.4.
  • Fix the Tests badge in the README to link to the workflow page (actions/workflows/test.yml) instead of the generic Actions tab, matching the convention used in other dblock repos.

Why

The CI matrix and pinned action versions were several years stale (Ruby 2.7/3.0 are EOL, actions/checkout@v3 is outdated). The Tests badge currently shows "no status" since this repo has no recorded workflow runs; merging this PR (and future pushes) will populate that history.

@dblock

dblock commented Aug 21, 2026

Copy link
Copy Markdown
Owner Author

Note: the danger (PR Linter) check fails with a pre-existing, unrelated issue — danger.yml has a hardcoded, now-expired/revoked personal access token (base64-encoded in the workflow file) that returns 401 Bad credentials. This predates this PR and isn't something I changed. Recommend replacing it with a proper repo secret (e.g. secrets.DANGER_GITHUB_API_TOKEN) in a follow-up.

Fixed in #49

dblock and others added 2 commits August 21, 2026 12:59
- Bump actions/checkout from v3 to v7 in all workflows.
- Update the test matrix to drop EOL Ruby 2.7/3.0/3.1 and add
  Ruby 3.3/3.4, keeping ruby-head/jruby-head as allowed failures.
- Bump the pinned Ruby version used by RuboCop and Danger jobs from
  3.1 to 3.4.
- Point the Tests badge link at the workflow page instead of the
  generic Actions tab, matching the convention used in other
  dblock repos.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dblock
dblock force-pushed the dblock/modernize-ci branch from 5af35fc to 6959117 Compare August 21, 2026 18:00
@dblock
dblock merged commit 6d258a2 into master Aug 21, 2026
17 checks passed
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