Skip to content

fix: explicitly specify main branch for hub client in migration utility#522

Merged
sirahd merged 2 commits intomainfrom
sirahd/fix_hub_client_jwt
Oct 3, 2025
Merged

fix: explicitly specify main branch for hub client in migration utility#522
sirahd merged 2 commits intomainfrom
sirahd/fix_hub_client_jwt

Conversation

@sirahd
Copy link
Collaborator

@sirahd sirahd commented Oct 3, 2025

The current hub client does not pass revision into the argument, which causes the moon-landing call to append create_pr=1 query param to the token API and returns 403 error.

@sirahd sirahd requested review from jsulz and seanses October 3, 2025 18:56
Copy link
Collaborator

@seanses seanses left a comment

Choose a reason for hiding this comment

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

LGTM!

@seanses
Copy link
Collaborator

seanses commented Oct 3, 2025

Root cause: repos disable Discussion and thus create_pr=1 query param fails pullRequestGuard.

@sirahd sirahd merged commit 89e5490 into main Oct 3, 2025
6 checks passed
@sirahd sirahd deleted the sirahd/fix_hub_client_jwt branch October 3, 2025 19:16
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.

3 participants