Skip to content

fix(skills): remove auto merge from create-pr default flow - #617

Merged
baleen37 merged 1 commit into
mainfrom
worktree/prancy-snuggling-pudding
Apr 15, 2026
Merged

fix(skills): remove auto merge from create-pr default flow#617
baleen37 merged 1 commit into
mainfrom
worktree/prancy-snuggling-pudding

Conversation

@baleen37

@baleen37 baleen37 commented Apr 15, 2026

Copy link
Copy Markdown
Owner

Summary

  • create-pr 스킬에서 gh pr merge --auto --squash를 기본 플로우에서 제거
  • auto merge는 사용자가 명시적으로 요청한 경우에만 실행되도록 변경

Changes

  • plugins/me/skills/create-pr/SKILL.md: auto merge와 wait-for-merge를 기본 플로우에서 주석 처리하고, 별도 조건부 섹션으로 분리

Tests

  • SKILL.md 구문 확인

Summary by CodeRabbit

  • Documentation
    • Updated PR creation skill merge workflow documentation. Auto-merge now requires explicit user request as a manual/conditional action, followed by the merge completion step. Previously, these steps executed automatically.

Auto merge should only run when explicitly requested by the user,
not as a default step in the PR creation workflow.
@baleen37
baleen37 enabled auto-merge (squash) April 15, 2026 02:32
@coderabbitai

coderabbitai Bot commented Apr 15, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d57d54b8-7361-45b9-8b2b-e3d7815ff91b

📥 Commits

Reviewing files that changed from the base of the PR and between 70ba9a1 and 449e01c.

📒 Files selected for processing (1)
  • plugins/me/skills/create-pr/SKILL.md

📝 Walkthrough

Walkthrough

The create-pr skill's merge workflow is modified to make auto-merge and wait-for-merge operations conditional and user-controlled instead of automatically executed. Previously active commands are commented out, with documentation added indicating these steps should only run upon explicit user request.

Changes

Cohort / File(s) Summary
Create-PR Skill Merge Configuration
plugins/me/skills/create-pr/SKILL.md
Auto-merge and wait-for-merge steps changed from automatic execution to conditional, user-triggered actions. Commands are commented out with added documentation specifying execution conditions and background execution mode for the wait script.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 Hop and pause, don't always merge so fast,
Let users choose when pull requests should pass,
With gentle comments and background care,
The workflow breathes with thoughtful flair! 🎯

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch worktree/prancy-snuggling-pudding

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@baleen37
baleen37 merged commit 4d2b84f into main Apr 15, 2026
4 of 5 checks passed
@baleen37
baleen37 deleted the worktree/prancy-snuggling-pudding branch April 15, 2026 02:33
baleen-release-app Bot pushed a commit that referenced this pull request Apr 15, 2026
## [16.7.4](v16.7.3...v16.7.4) (2026-04-15)

### Bug Fixes

* **skills:** remove auto merge from create-pr default flow ([#617](#617)) ([4d2b84f](4d2b84f))
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.

1 participant