Skip to content

Use rubyatscale/rubyfmt-action for the Rubyfmt CI check - #71

Merged
dduugg merged 1 commit into
mainfrom
use-rubyfmt-action
Aug 31, 2026
Merged

Use rubyatscale/rubyfmt-action for the Rubyfmt CI check#71
dduugg merged 1 commit into
mainfrom
use-rubyfmt-action

Conversation

@dduugg

@dduugg dduugg commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replaces the hand-rolled Install rubyfmt + Check formatting steps in the rubyfmt CI job with the newly published rubyatscale/rubyfmt-action.
  • Same check, same paths — just delegated to a maintained action instead of a copy-pasted install script. The action downloads a pinned, checksum-verified rubyfmt release (currently 0.14.2-10, vs. the 0.14.1 this repo was pinned to), and future version bumps happen centrally in that repo instead of needing a PR here each time.

Test plan

  • CI passes on this PR (the rubyfmt job should behave the same as before, modulo the newer rubyfmt version).

Replaces the manual 'download+install rubyfmt v0.14.1, then run
rubyfmt --check' steps with the published rubyatscale/rubyfmt-action
GitHub Action (https://github.com/rubyatscale/rubyfmt-action), which
downloads a pinned, checksum-verified rubyfmt release. This also
picks up newer rubyfmt releases automatically (the action currently
defaults to 0.14.2-10) instead of staying pinned to 0.14.1.
@dduugg
dduugg requested a review from a team as a code owner August 31, 2026 23:05
@github-project-automation github-project-automation Bot moved this to Triage in Modularity Aug 31, 2026
@dduugg
dduugg merged commit b11c01c into main Aug 31, 2026
10 checks passed
@dduugg
dduugg deleted the use-rubyfmt-action branch August 31, 2026 23:18
@github-project-automation github-project-automation Bot moved this from Triage to Done in Modularity Aug 31, 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.

1 participant