Skip to content

Commit 3084835

Browse files
committed
fix(bugfix): normalize BRANCH_NAME placeholder and add code fence label
Address CodeRabbit review feedback on PR #79: - Make BRANCH_NAME the full branch name (e.g. bugfix/issue-42-null-check) and remove redundant bugfix/ prefix from all consumers to prevent double-prefixing like bugfix/bugfix/... - Add language label to pre-flight gate fenced code block (MD040) Made-with: Cursor
1 parent be60dd2 commit 3084835

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

  • workflows/bugfix/.claude/skills/pr

workflows/bugfix/.claude/skills/pr/SKILL.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ and tell the user.
174174
table.** Every row must have a value or an explicit "unknown". If you cannot
175175
fill in a row, that itself is important information that determines the flow.
176176

177-
```
177+
```text
178178
Pre-flight summary:
179179
| Placeholder | Value |
180180
| -------------------- | ------------------ |
@@ -392,7 +392,7 @@ git rebase fork/main
392392
### Step 5: Create a Branch
393393

394394
```bash
395-
git checkout -b bugfix/BRANCH_NAME
395+
git checkout -b BRANCH_NAME
396396
```
397397

398398
Branch naming conventions:
@@ -435,7 +435,7 @@ and include that instead.
435435
gh auth setup-git
436436

437437
# Push the branch
438-
git push -u fork bugfix/BRANCH_NAME
438+
git push -u fork BRANCH_NAME
439439
```
440440

441441
**If this fails:**
@@ -464,7 +464,7 @@ on some repos):
464464
gh pr create \
465465
--draft \
466466
--repo UPSTREAM_OWNER/REPO \
467-
--head FORK_OWNER:bugfix/BRANCH_NAME \
467+
--head FORK_OWNER:BRANCH_NAME \
468468
--base main \
469469
--title "fix(SCOPE): SHORT_DESCRIPTION" \
470470
--body-file artifacts/bugfix/docs/pr-description.md
@@ -486,7 +486,7 @@ Do NOT retry, do NOT debug further, do NOT fall back to a patch file. Instead:
486486
query parameters so the PR form opens fully populated:
487487

488488
```text
489-
https://github.com/UPSTREAM_OWNER/REPO/compare/main...FORK_OWNER:bugfix/BRANCH_NAME?expand=1&title=URL_ENCODED_TITLE&body=URL_ENCODED_BODY
489+
https://github.com/UPSTREAM_OWNER/REPO/compare/main...FORK_OWNER:BRANCH_NAME?expand=1&title=URL_ENCODED_TITLE&body=URL_ENCODED_BODY
490490
```
491491

492492
URL-encode the title and body. If the encoded URL would exceed ~8KB
@@ -513,7 +513,7 @@ Do NOT retry, do NOT debug further, do NOT fall back to a patch file. Instead:
513513
```
514514

515515
**If "branch not found"**: The push in Step 7 may have failed silently.
516-
Verify with `git ls-remote fork bugfix/BRANCH_NAME`.
516+
Verify with `git ls-remote fork BRANCH_NAME`.
517517

518518
### Step 9: Confirm and Report
519519

0 commit comments

Comments
 (0)