Unverified 提交 30ed10aa authored 作者: wwwillchen-bot's avatar wwwillchen-bot 提交者: GitHub

Add step to remove needs-human:review-issue label after push (#2654)

## Summary - Updates both `pr-push.md` and `fast-push.md` commands to automatically remove the `needs-human:review-issue` label after pushing - This label indicates the issue needed human review before work started, which is now complete once the PR is pushed ## Test plan - Run `/dyad:pr-push` on a PR with the `needs-human:review-issue` label and verify it gets removed - Run `/dyad:fast-push` and verify the same behavior 🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2654" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Automatically removes the needs-human:review-issue label after pushing in pr-push and fast-push, so issues reflect that human review is complete. Keeps PRs and labels in sync without manual cleanup. - **New Features** - pr-push and fast-push run gh pr edit --remove-label "needs-human:review-issue" after push, gracefully handling cases where the label doesn’t exist. <sup>Written for commit 5d9bd27c187502459b83bfde12b960a0f89c5cd5. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> Co-authored-by: 's avatarWill Chen <willchen90@gmail.com> Co-authored-by: 's avatarClaude Opus 4.5 <noreply@anthropic.com>
上级 42531151
......@@ -162,6 +162,13 @@ You MUST use the TaskCreate and TaskUpdate tools to track your progress. At the
gh pr edit --add-label "cc:request"
```
**Remove review-issue label:**
After pushing, remove the `needs-human:review-issue` label if it exists (this label indicates the issue needed human review before work started, which is now complete):
```
gh pr edit --remove-label "needs-human:review-issue" 2>/dev/null || true
```
7. **Summarize the results:**
- Report if a new feature branch was created (and its name)
- Report any uncommitted changes that were committed in step 2
......
......@@ -170,12 +170,21 @@ Commit any uncommitted changes, run lint checks, fix any issues, and push the cu
Use the commit messages and changed files to write a good title and summary.
9. **Summarize the results:**
- Report if a new feature branch was created (and its name)
- Report any uncommitted changes that were committed in step 2
- Report any files that were IGNORED and not committed (if any), explaining why they were skipped
- Report any lint fixes that were applied
- Confirm tests passed
- Confirm the branch has been pushed
- Report any learnings added to `AGENTS.md`
- **Include the PR URL** (either newly created or existing)
9. **Remove review-issue label:**
After pushing, remove the `needs-human:review-issue` label if it exists (this label indicates the issue needed human review before work started, which is now complete):
```
gh pr edit --remove-label "needs-human:review-issue" 2>/dev/null || true
```
10. **Summarize the results:**
- Report if a new feature branch was created (and its name)
- Report any uncommitted changes that were committed in step 2
- Report any files that were IGNORED and not committed (if any), explaining why they were skipped
- Report any lint fixes that were applied
- Confirm tests passed
- Confirm the branch has been pushed
- Report any learnings added to `AGENTS.md`
- **Include the PR URL** (either newly created or existing)
Markdown 格式
0%
您添加了 0 到此讨论。请谨慎行事。
请先完成此评论的编辑!
注册 或者 后发表评论