Skip to content

fix(git): remove -u flag from push instruction#62

Merged
cblecker merged 1 commit into
mainfrom
fix/git-remove-push-u-flag
Jun 26, 2026
Merged

fix(git): remove -u flag from push instruction#62
cblecker merged 1 commit into
mainfrom
fix/git-remove-push-u-flag

Conversation

@cblecker

@cblecker cblecker commented Jun 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • Remove -u flag from the git push instruction in the PR creation workflow
  • The -u flag causes errors in the Claude Code sandbox environment

Test plan

  • Run bash git/scripts/git-instructions.sh and confirm output says "Push to remote if needed" without -u flag
  • Validate plugin: claude plugin validate ./git

Summary by CodeRabbit

  • Chores
    • Updated the plugin version from 1.5.0 to 1.5.1.
    • Refined the pull request workflow guidance to simplify push instructions.

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
Copilot AI review requested due to automatic review settings June 26, 2026 16:49
@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f6818c19-c20b-45e4-8838-c8a437aba24e

📥 Commits

Reviewing files that changed from the base of the PR and between 0873acc and 44bd975.

📒 Files selected for processing (2)
  • git/.claude-plugin/plugin.json
  • git/scripts/git-instructions.sh

📝 Walkthrough

Walkthrough

The 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 -u push flag guidance.

Changes

Plugin Release Metadata

Layer / File(s) Summary
Version bump
git/.claude-plugin/plugin.json
version changes from 1.5.0 to 1.5.1.

Creating Pull Request Instructions

Layer / File(s) Summary
Push step wording
git/scripts/git-instructions.sh
The “Creating pull requests” step changes the push instruction to remove the -u upstream flag wording.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

I nibbled the manifest, tidy and light,
then hopped through the script to set things right.
A smaller push, a version new,
thump-thump—my little paws approve! 🐇

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: removing the -u flag from the git push instruction.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/git-remove-push-u-flag

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.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.0 to 1.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.

@cblecker cblecker merged commit 0c4d493 into main Jun 26, 2026
13 checks passed
@cblecker cblecker deleted the fix/git-remove-push-u-flag branch June 26, 2026 16:53
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.

2 participants