Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork398
Fix temporaryDeploymentBranch conflict by replacing slash with hyphen#1892
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:dev
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
@JamesIves 👋 This repository doesn't haveCopilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs. I can generate a .github/copilot-instructions.md file for you automatically. Clickhere to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review. |
Co-authored-by: JamesIves <10888441+JamesIves@users.noreply.github.com>
Co-authored-by: JamesIves <10888441+JamesIves@users.noreply.github.com>
github-pages-deploy-action branch due to conflicttemporaryDeploymentBranch
Uh oh!
There was an error while loading.Please reload this page.
This PR fixes a Git reference conflict that occurs when creating temporary deployment branches. The action was failing when a branch named
github-pages-deploy-actionalready exists in the repository.Problem
The action creates temporary branches using the pattern:
When a branch named
github-pages-deploy-actionalready exists, Git cannot create the temporary branch because it interprets branch names with forward slashes as nested references. This causes the error:Solution
Changed the temporary branch naming pattern from:
`github-pages-deploy-action/${randomString}`To:
`github-pages-deploy-action-${randomString}`This uses a hyphen separator instead of a forward slash, eliminating the Git reference conflict while maintaining a clear naming convention.
Changes Made
temporaryDeploymentBranchTesting
This minimal change resolves the conflict without affecting any other functionality or breaking existing workflows.
Fixes#1712.
✨ Let Copilot coding agentset things up for you — coding agent works faster and does higher quality work when set up for your repo.