Skip to content

[codex] Fix release-hold auth and timestamp parsing#10

Draft
jleni wants to merge 1 commit intomainfrom
codex/fix-release-hold-auth-portability
Draft

[codex] Fix release-hold auth and timestamp parsing#10
jleni wants to merge 1 commit intomainfrom
codex/fix-release-hold-auth-portability

Conversation

@jleni
Copy link
Copy Markdown
Member

@jleni jleni commented Apr 16, 2026

What changed

  • export GH_TOKEN to the composite step so gh commands authenticate in GitHub Actions
  • replace GNU-only date -d timestamp parsing with a portable Python-based ISO-8601 conversion

Why

The new release-hold action had two portability issues:

  • gh was invoked without an auth token, which can fail in CI and private repositories
  • date -d only works on GNU date, so the action would fail on macOS runners

Impact

  • release-hold now works with authenticated gh calls by default in GitHub Actions
  • timestamp parsing is portable across GitHub-hosted runners that provide Python

Validation

  • parsed release-hold/action.yml successfully with Ruby YAML
  • verified the Python timestamp conversion works with a sample ISO-8601 timestamp
  • reviewed the resulting git diff for the targeted fix scope

@jleni jleni requested a review from emmanuelm41 April 16, 2026 14:37
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