Skip to content

release: Use trusted publishing for RubyGems releases#282

Merged
jbeckwith-oai merged 1 commit into
nextfrom
codex/trusted-rubygems-publishing
Jun 4, 2026
Merged

release: Use trusted publishing for RubyGems releases#282
jbeckwith-oai merged 1 commit into
nextfrom
codex/trusted-rubygems-publishing

Conversation

@jbeckwith-oai

Copy link
Copy Markdown
Contributor

Summary

  • switch RubyGems publishing to the official rubygems/release-gem action using GitHub OIDC trusted publishing
  • remove the old secret-based manual publish workflow and helper script
  • build release gems into pkg/ so the RubyGems action can wait for the published artifact
  • require RubyGems MFA for future gem releases

Validation

  • ruby -e 'require "yaml"; YAML.load_file(".github/workflows/create-releases.yml"); puts "workflow yaml ok"'
  • ruby -c Rakefile
  • bundle _2.4.1_ exec rake build:gem

Notes

rubygems/release-gem is pinned to the SHA for latest release v1.2.0 (6317d8d1f7e28c24d28f6eff169ea854948bd9f7). The direct branch push bypassed the repository workflow-file rule after the repository settings were updated.

@jbeckwith-oai jbeckwith-oai changed the title [codex] Use trusted publishing for RubyGems releases release: Use trusted publishing for RubyGems releases Jun 4, 2026
@jbeckwith-oai jbeckwith-oai marked this pull request as ready for review June 4, 2026 15:52
@jbeckwith-oai jbeckwith-oai requested a review from a team as a code owner June 4, 2026 15:52

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f56390fa9e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

# `RUBYGEMS_HOST` is only required for private gem repositories, not https://rubygems.org
RUBYGEMS_HOST: ${{ secrets.OPENAI_RUBYGEMS_HOST || secrets.RUBYGEMS_HOST }}
GEM_HOST_API_KEY: ${{ secrets.OPENAI_GEM_HOST_API_KEY || secrets.GEM_HOST_API_KEY }}
uses: rubygems/release-gem@6317d8d1f7e28c24d28f6eff169ea854948bd9f7 # v1.2.0

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Preserve a retry path for failed gem publishes

With .github/workflows/publish-gem.yml deleted, this guarded publish step is now the only way to push the gem. If RubyGems publishing fails after trigger-release-please has already created the GitHub release/tag, rerunning or manually dispatching this workflow will run release-please again and releases_created will be false, so the gem publish step is skipped and maintainers no longer have a workflow path to retry the missing RubyGems release. Consider adding a trusted-publishing manual retry path or a workflow_dispatch override for this step.

Useful? React with 👍 / 👎.

@jbeckwith-oai jbeckwith-oai force-pushed the codex/trusted-rubygems-publishing branch from f56390f to 0fd117b Compare June 4, 2026 16:01
@apcha-oai apcha-oai changed the base branch from main to next June 4, 2026 16:05

@apcha-oai apcha-oai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM pointed to next so we can try this for next minor release

@jbeckwith-oai jbeckwith-oai merged commit 4f1bde8 into next Jun 4, 2026
11 checks passed
stainless-app Bot added a commit that referenced this pull request Jun 4, 2026
* Use trusted publishing for RubyGems releases (#282)

* release: 0.66.1

---------

Co-authored-by: Justin Beckwith <jbeckwith@openai.com>
Co-authored-by: stainless-app[bot] <142633134+stainless-app[bot]@users.noreply.github.com>
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