Skip to content

Pin setup-rye in GitHub workflows#3083

Merged
apcha-oai merged 1 commit intonextfrom
dev/codex/pin-setup-rye-workflows
Apr 14, 2026
Merged

Pin setup-rye in GitHub workflows#3083
apcha-oai merged 1 commit intonextfrom
dev/codex/pin-setup-rye-workflows

Conversation

@mcgrew-oai
Copy link
Copy Markdown

Changes being requested

Replace manual Rye installation in GitHub Actions workflows with eifinger/setup-rye pinned to a full commit SHA.

This updates the Rye setup steps in:

  • .github/workflows/ci.yml
  • .github/workflows/create-releases.yml
  • .github/workflows/detect-breaking-changes.yml
  • .github/workflows/publish-pypi.yml

Changes included:

  • replace curl -sSf https://rye.astral.sh/get | bash bootstrap steps
  • pin eifinger/setup-rye to c694239a43768373e87d0103d7f547027a23f3c8
  • keep Rye explicitly pinned to 0.44.0
  • enable action-level caching
  • preserve existing workflow conditions
  • set working-directory: openai-python for the nested checkout in detect-breaking-changes.yml

Additional context & links

Verified locally that:

  • there are no remaining manual Rye install steps in .github/workflows
  • there are no remaining RYE_VERSION / RYE_INSTALL_OPTION env blocks for Rye setup
  • the edited workflow files parse as YAML

Upstream references used for the pin/version:

  • eifinger/setup-rye latest release: v4.2.9
  • release commit: c694239a43768373e87d0103d7f547027a23f3c8
  • astral-sh/rye latest release: 0.44.0

@mcgrew-oai mcgrew-oai marked this pull request as ready for review April 13, 2026 23:55
@mcgrew-oai mcgrew-oai requested a review from a team as a code owner April 13, 2026 23:55
@apcha-oai apcha-oai changed the base branch from main to next April 14, 2026 14:04
@apcha-oai apcha-oai merged commit 0bf10b3 into next Apr 14, 2026
10 checks passed
@apcha-oai apcha-oai deleted the dev/codex/pin-setup-rye-workflows branch April 14, 2026 14:06
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