Skip to content

Update CI runners for scala2 branch#632

Merged
adamw merged 3 commits intoscala2from
update-ci-runners
Feb 24, 2026
Merged

Update CI runners for scala2 branch#632
adamw merged 3 commits intoscala2from
update-ci-runners

Conversation

@adamw
Copy link
Copy Markdown
Member

@adamw adamw commented Feb 24, 2026

Summary

  • Replace inline ci job with reusable build-scala.yml workflow (matches scala3 branch)
  • Update mima and publish runners from discontinued ubuntu-20.04 to ubuntu-22.04
  • Add label and auto-merge jobs for Scala Steward PRs (matches scala3 branch)

Fixes the "maximum execution time while awaiting a runner" issue reported in #628.

🤖 Generated with Claude Code

Port CI changes from scala3 branch: use reusable build workflow,
update runners to ubuntu-22.04, add label and auto-merge jobs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 24, 2026 16:38
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR modernizes the CI workflow for the scala2 branch by adopting reusable workflows and updating to supported GitHub runner versions. The changes align the scala2 branch CI configuration with the scala3 branch, addressing runner availability issues that were causing build failures.

Changes:

  • Replaced inline ci job with centralized build-scala.yml reusable workflow from softwaremill/github-actions-workflows
  • Updated mima and publish job runners from deprecated ubuntu-20.04 to ubuntu-22.04
  • Added label and auto-merge jobs for automated handling of Scala Steward dependency update PRs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 64 to 66
uses: actions/setup-java@v4
with:
distribution: 'temurin'
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The matrix.java variable is referenced here, but no matrix strategy is defined in the publish job. This will cause the workflow to fail because the variable is undefined. You should either define a matrix strategy or replace ${{ matrix.java }} with a specific Java version (e.g., '21' or '17').

Copilot uses AI. Check for mistakes.
adamw and others added 2 commits February 24, 2026 18:04
- Replace undefined matrix.java with explicit '17' in mima and publish jobs
- Exclude .sbt/matrix/ generated files from scalafmtCheckAll

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The reusable build-scala.yml workflow runs scalafmtCheckAll which is
incompatible with sbt-projectmatrix generated files. Revert to inline
steps (compile + test) but with updated runners and java-version.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@adamw adamw merged commit 9dd1209 into scala2 Feb 24, 2026
10 checks passed
@adamw adamw deleted the update-ci-runners branch February 24, 2026 17: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