Skip to content

Fix generated PR branch collision - #2125

Merged
albanD merged 1 commit into
sitefrom
cc_fix_generated_pr_branch_collision
Jul 27, 2026
Merged

Fix generated PR branch collision#2125
albanD merged 1 commit into
sitefrom
cc_fix_generated_pr_branch_collision

Conversation

@albanD

@albanD albanD commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Author Notes

Fix the two workflows clobbering each other, leading to empty PRs like #2124.

Agent Notes

AI-generated summary:

Both create-pull-request jobs used the default create-pull-request/patch branch, allowing one workflow job to overwrite the other job's changes and close its PR as empty. This change assigns each job a distinct, stable branch.

Validation:

  • git diff --check
  • YAML syntax parsed successfully with Python and PyYAML.

Both create-pull-request jobs relied on the action's default branch, so the later job force-pushed over the earlier job's generated changes and closed its PR. Give each job a stable, distinct branch so they can maintain independent pull requests.

Test Plan:

```
git diff --check
python -c "import yaml; yaml.compose(open('\.github/workflows/update-quick-start-module.yml')); print('YAML syntax OK')"
```

This commit was authored with an AI assistant.
@netlify

netlify Bot commented Jul 27, 2026

Copy link
Copy Markdown

Deploy Preview for pytorch-dot-org-preview ready!

Name Link
🔨 Latest commit 38f0f38
🔍 Latest deploy log https://app.netlify.com/projects/pytorch-dot-org-preview/deploys/6a67774ba608f800089530c5
😎 Deploy Preview https://deploy-preview-2125--pytorch-dot-org-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@albanD
albanD marked this pull request as ready for review July 27, 2026 15:21
@albanD
albanD merged commit a665fcd into site Jul 27, 2026
16 of 18 checks passed
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.

2 participants