Skip to content

Resolve CI annotation warnings#170

Merged
dduugg merged 3 commits into
mainfrom
fix-ci-annotations-warnings
Jun 10, 2026
Merged

Resolve CI annotation warnings#170
dduugg merged 3 commits into
mainfrom
fix-ci-annotations-warnings

Conversation

@dduugg

@dduugg dduugg commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Resolves the annotation warnings surfaced in CI (run 27302012478).

Warnings addressed

1. Unexpected input(s) 'ruby-version-file'

Neither ruby/setup-ruby@v1 nor oxidize-rb/actions/setup-ruby-and-rust@v1 accepts a ruby-version-file input (verified against each action's action.yml; the warning's valid-input list matches ruby/setup-ruby exactly). Both actions automatically read .ruby-version when ruby-version is unset, so the input was a no-op that only produced a warning.

Removed the three usages:

  • ci.ymlrspec_ruby_version_file job (setup-ruby-and-rust)
  • ci.ymlstatic_type_check job (ruby/setup-ruby)
  • cd.ymlrelease job (setup-ruby-and-rust)

Behavior is unchanged: each step still resolves Ruby from .ruby-version via auto-detection.

2. Node.js 20 actions are deprecated … actions/checkout@v4

Bumped all 5 occurrences of actions/checkout to @v6, which runs on Node.js 24. The other actions in these workflows already run on Node.js 24.

Testing

Both workflow files validated as parseable YAML.

- Remove unsupported ruby-version-file input from setup-ruby steps
  (both ruby/setup-ruby and oxidize-rb/setup-ruby-and-rust auto-read
  .ruby-version when ruby-version is unset)
- Bump actions/checkout v4 -> v5 to run on Node.js 24
@dduugg dduugg requested a review from a team as a code owner June 10, 2026 19:56
@github-project-automation github-project-automation Bot moved this to Triage in Modularity Jun 10, 2026
@dduugg dduugg enabled auto-merge (squash) June 10, 2026 20:00

@conwayje conwayje left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lg!

@dduugg dduugg merged commit d861735 into main Jun 10, 2026
10 checks passed
@dduugg dduugg deleted the fix-ci-annotations-warnings branch June 10, 2026 20:29
@github-project-automation github-project-automation Bot moved this from Triage to Done in Modularity Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants