Skip to content

Conversation

bolinfest
Copy link
Collaborator

@bolinfest bolinfest commented Sep 6, 2025

The advantage of this implementation is that it can be run from "anywhere" so long as the user has gh installed with the appropriate credentials to write to the openai/codex repo. Unlike the previous implementation, it avoids the overhead of creating a local clone of the repo.

Ran:

./codex-rs/scripts/create_github_release 0.31.0-alpha.2

which appeared to work as expected:


Stack created with Sapling. Best reviewed with ReviewStack.

print("Fetching branch head...")
base_commit = get_branch_head()
print(f"Base commit: {base_commit}")
print("Fetching commit tree...")
Copy link
Collaborator

Choose a reason for hiding this comment

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

[non-blocking] Should the start logs be in the functions themselves? Or do we have. a preference for keeping the functions clean?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think once we start using it, we'll decide.

Honestly, I put these here originally because I had to ask Codex to print-debug itself :P

bolinfest added a commit that referenced this pull request Sep 6, 2025
…3226)

Migrating to Python to make this script easier to iterate on.

---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/openai/codex/pull/3226).
* #3231
* #3230
* #3228
* __->__ #3226
bolinfest added a commit that referenced this pull request Sep 6, 2025
… directory (#3228)

Ran:

```
./codex-rs/scripts/create_github_release 0.31.0-alpha.1
```

which appeared to work as expected:

- workflow https://github.com/openai/codex/actions/runs/17508403922
- release
https://github.com/openai/codex/releases/tag/rust-v0.31.0-alpha.1

---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/openai/codex/pull/3228).
* #3231
* #3230
* __->__ #3228
* #3226
Base automatically changed from pr3228 to main September 6, 2025 04:57
@bolinfest bolinfest merged commit b1d5f7c into main Sep 6, 2025
34 checks passed
@bolinfest bolinfest deleted the pr3230 branch September 6, 2025 04:58
@github-actions github-actions bot locked and limited conversation to collaborators Sep 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants