fix(git): remove -u flag from push instruction#62
Conversation
The -u flag on git push causes errors in the Claude Code sandbox. Remove it from the PR creation instructions. Assisted-by: Claude:claude-opus-4-6
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe plugin manifest version is bumped from 1.5.0 to 1.5.1, and the git instructions update the pull-request workflow wording to remove the ChangesPlugin Release Metadata
Creating Pull Request Instructions
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
This PR removes the -u (set-upstream) flag from the git push instruction in the git plugin's dynamically generated pull request creation workflow. According to the description, the -u flag causes errors in the Claude Code sandbox environment. This aligns with the plugin's existing branch.autosetupmerge=false override, which intentionally avoids tracking setup. The patch version is bumped accordingly.
Changes:
- Changed the push step text from "Push to remote with -u flag if needed" to "Push to remote if needed" in
git-instructions.sh. - Bumped the git plugin version from
1.5.0to1.5.1.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| git/scripts/git-instructions.sh | Removes the -u flag from the push instruction in the PR creation steps |
| git/.claude-plugin/plugin.json | Patch version bump (1.5.0 → 1.5.1) reflecting the fix |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
-uflag from thegit pushinstruction in the PR creation workflow-uflag causes errors in the Claude Code sandbox environmentTest plan
bash git/scripts/git-instructions.shand confirm output says "Push to remote if needed" without-u flagclaude plugin validate ./gitSummary by CodeRabbit