Skip to content

Migrate CI to github-actions@v2; drop wrapper cron - #221

Merged
yonatankarp merged 1 commit into
mainfrom
migrate-to-v2
May 25, 2026
Merged

Migrate CI to github-actions@v2; drop wrapper cron#221
yonatankarp merged 1 commit into
mainfrom
migrate-to-v2

Conversation

@yonatankarp

@yonatankarp yonatankarp commented May 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • Bump shared workflow refs to `@v2`
  • Drop unused `context` / `build_dockerfile` inputs (v2 defaults match)
  • Grant `pull-requests:write` + `checks:write` for v2's `publish-test-reports`
  • Delete `update-gradle-wrapper.yml` cron — Dependabot's gradle ecosystem now covers the wrapper

Branch protection's `linters / Linters` context renamed to `linters / linters` to match v2's new check name (the v2 linters job dropped the friendly name; check key matches the job id).

Test plan

  • CI runs green
  • Test Results check shows under "Test Results (openapi-usage-example)"

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated CI workflow dependencies to newer versions
    • Modified pipeline job permissions configuration
    • Removed automated Gradle wrapper update workflow

Review Change Stack

- @v1@v2 across ci.yml, dependabot-auto-merge.yml, linters.yml
- Drop unused inputs (context, build_dockerfile) — v2 defaults match
- Add pull-requests:write + checks:write on the pipeline job (required
  by v2's publish-test-reports step)
- Delete update-gradle-wrapper.yml — Dependabot's gradle ecosystem
  covers the wrapper natively now

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented May 25, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The PR upgrades three reusable workflow references in the pull request CI workflow from @v1 to @v2: the main ci.yml, dependabot-auto-merge.yml, and linters.yml. The pipeline job adds explicit permissions (including checks: write) and removes prior input configuration. The unused Gradle wrapper workflow file is deleted entirely.

Changes

Reusable Workflow Version Upgrades

Layer / File(s) Summary
Workflow dependency v1 to v2 upgrades
.github/workflows/pull_request.yml, .github/workflows/update-gradle-wrapper.yml
ci.yml, dependabot-auto-merge.yml, and linters.yml reusable workflows are updated from @v1 to @v2 in the pull request workflow. The pipeline job adds explicit permissions (including checks: write) and removes context: . and build_dockerfile: false input parameters. The Gradle wrapper workflow is deleted (12 lines).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Poem

🐰 A hop and a skip through versions new,
From v1 to v2, the workflows flew,
With checks now written and permissions clear,
The CI pipeline's refined without fear!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Migrate CI to github-actions@v2; drop wrapper cron' directly reflects the two main changes: upgrading shared workflows to v2 and removing the Gradle wrapper update cron job.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch migrate-to-v2

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@yonatankarp
yonatankarp enabled auto-merge (rebase) May 25, 2026 11:20
@github-actions

Copy link
Copy Markdown

Test Results (openapi-usage-example)

4 tests   4 ✅  3s ⏱️
2 suites  0 💤
2 files    0 ❌

Results for commit de60154.

@yonatankarp
yonatankarp merged commit 132c8a2 into main May 25, 2026
4 of 5 checks passed
@yonatankarp
yonatankarp deleted the migrate-to-v2 branch May 25, 2026 11:22
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