Skip to content

[ci] allow claude to open PRs for certain instructions.#13536

Open
sayakpaul wants to merge 5 commits intomainfrom
fix-claude-workflow-post
Open

[ci] allow claude to open PRs for certain instructions.#13536
sayakpaul wants to merge 5 commits intomainfrom
fix-claude-workflow-post

Conversation

@sayakpaul
Copy link
Copy Markdown
Member

What does this PR do?

Potentially fixes #13472 (comment)

@sayakpaul sayakpaul requested a review from yiyixuxu April 22, 2026 00:37
@github-actions github-actions Bot added CI size/M PR with diff < 200 LOC labels Apr 22, 2026
@sayakpaul sayakpaul requested a review from paulinebm April 22, 2026 00:38
@github-actions github-actions Bot added size/M PR with diff < 200 LOC and removed size/M PR with diff < 200 LOC labels Apr 22, 2026
@github-actions github-actions Bot added size/M PR with diff < 200 LOC and removed size/M PR with diff < 200 LOC labels Apr 22, 2026
@github-actions github-actions Bot added size/M PR with diff < 200 LOC and removed size/M PR with diff < 200 LOC labels Apr 22, 2026
Copy link
Copy Markdown
Collaborator

@yiyixuxu yiyixuxu left a comment

Choose a reason for hiding this comment

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

thanks!
i left a comment,

CLAUDE_COMMIT=$(git rev-parse HEAD)

git fetch --depth=1 origin "$BASE_BRANCH"
git switch -c "$NEW_BRANCH" "origin/$BASE_BRANCH"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ohhh i think, if we alreeady at a claude PR -> it's better if we stay there until we merge no?
so basically if i said COMMIT this from a contributor's PR -> claude open a new PR -> we iterate there where claude can directly commit into its own PR -> merge

@github-actions github-actions Bot added size/M PR with diff < 200 LOC and removed size/M PR with diff < 200 LOC labels Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI size/M PR with diff < 200 LOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants