Skip to content

Fix release workflow: add workflow_dispatch, environment, and .tool-versions#1818

Merged
neilvcarvalho merged 2 commits intomainfrom
nc/fix-release
May 4, 2026
Merged

Fix release workflow: add workflow_dispatch, environment, and .tool-versions#1818
neilvcarvalho merged 2 commits intomainfrom
nc/fix-release

Conversation

@neilvcarvalho
Copy link
Copy Markdown
Member

@neilvcarvalho neilvcarvalho commented May 4, 2026

Summary

  • Adds workflow_dispatch to the release workflow so a release can be triggered manually without needing to bump lib/factory_bot/version.rb
  • Runs the release job in the release GitHub Actions environment, enabling environment-scoped secrets and optional required reviewers
  • Adds a .tool-versions file (Ruby 4.0) and removes hardcoded ruby-version values from setup-ruby steps in release.yml, prepare-release.yml, and the standard job in build.yml — the version is now defined in one place

The build matrix job is unchanged since it iterates over explicit Ruby versions by design.

Test plan

  • Verify the Release workflow appears with a "Run workflow" button in the Actions tab after merging
  • Confirm CI workflows pick up Ruby from .tool-versions correctly

neilvcarvalho and others added 2 commits May 4, 2026 17:04
Replaces hardcoded ruby-version values in setup-ruby steps with
.tool-versions so the Ruby version is defined in one place.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Adds workflow_dispatch so a release can be triggered without bumping
version.rb. Also runs the job in the "release" GitHub Actions environment
for environment-scoped secrets and required reviewers.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@neilvcarvalho neilvcarvalho requested a review from vburzynski as a code owner May 4, 2026 20:05
@neilvcarvalho neilvcarvalho merged commit 8b2d41a into main May 4, 2026
28 checks passed
@neilvcarvalho neilvcarvalho deleted the nc/fix-release branch May 4, 2026 20:07
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