Skip to content

fix(ci): give the sync push step the token its git credential helper reads - #18

Merged
onmax merged 1 commit into
mainfrom
fix/sync-upstream-push-token
Nov 12, 2026
Merged

onmax merged 1 commit into
mainfrom
fix/sync-upstream-push-token

Conversation

@onmax

@onmax onmax commented Nov 12, 2026

Copy link
Copy Markdown
Member

The daily "Sync upstream" run has failed since 2026-11-11, after the merge and all checks pass. The "Advance verified fork" step fails with could not read Username for 'https://github.com' (run).

gh auth setup-git in the first step makes git ask gh for credentials, and gh reads GH_TOKEN. Only the first step and the publish step set it, so the push has no token.

This sets GH_TOKEN on the push step too.

Prepared with Claude Opus 5.5 in T3 Code.

@chatgpt-codex-connector

Copy link
Copy Markdown
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ❌ Failed 2026-11-12T17:57:44.261454Z
We couldn’t complete this review. Comment @codex review to try again.
ed4630c PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS labels Nov 12, 2026
@onmax
onmax merged commit 30a6d6c into main Nov 12, 2026
10 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant