Skip to content

Adopt shared Scala CI workflow - #26

Merged
mr-git merged 4 commits into
masterfrom
adopt-shared-ci
Aug 13, 2026
Merged

Adopt shared Scala CI workflow#26
mr-git merged 4 commits into
masterfrom
adopt-shared-ci

Conversation

@stasimus

@stasimus stasimus commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Replaces the hand-written ci.yml with a call to the shared workflow, so tests, coverage, binary compatibility, formatting and scaladoc are configured centrally. Coverage stays off, as before. Binary compatibility is switched off explicitly because the repo has no sbt-version-policy; it was never enforced.

Checks now run as explicit sbt tasks rather than through the check alias, and checkout is unshallow so versionPolicyCheck has a previous version to compare against.

Part of evolution-gaming/scala-github-actions#5

Summary by CodeRabbit

  • Chores
    • Streamlined continuous integration workflow configuration.
    • Updated automated build checks to use a shared workflow.
    • Disabled coverage and binary compatibility checks.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 0f70edec-fe1f-4dee-af96-eea058128f20

📥 Commits

Reviewing files that changed from the base of the PR and between 440b1e9 and 1cf2f36.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

The CI workflow now uses evolution-gaming/scala-github-actions for shared execution. It disables coverage and binary compatibility checks and condenses the push branch configuration.

Changes

CI workflow unification

Layer / File(s) Summary
Adopt shared CI execution
.github/workflows/ci.yml
The workflow delegates runner setup, Scala version handling, caching, build, and test commands to evolution-gaming/scala-github-actions. Coverage and binary compatibility checks are disabled.

Estimated code review effort: 2 (Simple) | ~10 minutes

Mergeability Score: ⚪ Minimal · up to 1cf2f

This change centralizes Scala CI configuration without any supplied evidence of a current correctness or production-impact risk; no actionable merge-blocking risk remains beyond normal checks and review.

Possibly related issues

  • evolution-gaming/scala-github-actions#5 — The workflow change implements the issue’s CI unification objective by adopting the shared pipeline.
🚥 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 clearly and concisely describes the main change: adopting the shared Scala CI workflow.
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.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch adopt-shared-ci

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

@mr-git
mr-git merged commit fbc389a into master Aug 13, 2026
5 of 6 checks passed
@mr-git
mr-git deleted the adopt-shared-ci branch August 13, 2026 09:17
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