Conversation
|
🧙 Sourcery has finished reviewing your pull request! Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
Adds a new Fork (Git client) custom command to update the current branch from multiple remotes, aligning with the existing “Fetch All”, “Push All”, and “Delete All” workflow in .fork/custom-commands.json.
Changes:
- Added a “Pull All” custom command that stashes local changes and performs rebasing pulls from
origin,gitee.com, andgitcode.com.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| }, | ||
| { | ||
| "action" : { | ||
| "script" : "git stash save --message \"Pull autostash\"\ngit pull --rebase=true origin --prune\ngit pull --rebase=true gitee.com\ngit pull --rebase=true gitcode.com", |
There was a problem hiding this comment.
git pull --rebase=true origin --prune passes --prune as a refspec (because it comes after the remote), so pruning won’t run and the command may fail. Move --prune before the remote name (e.g., git pull --rebase --prune origin).
| "script" : "git stash save --message \"Pull autostash\"\ngit pull --rebase=true origin --prune\ngit pull --rebase=true gitee.com\ngit pull --rebase=true gitcode.com", | |
| "script" : "git stash save --message \"Pull autostash\"\ngit pull --rebase=true --prune origin\ngit pull --rebase=true gitee.com\ngit pull --rebase=true gitcode.com", |
| }, | ||
| { | ||
| "action" : { | ||
| "script" : "git stash save --message \"Pull autostash\"\ngit pull --rebase=true origin --prune\ngit pull --rebase=true gitee.com\ngit pull --rebase=true gitcode.com", |
There was a problem hiding this comment.
The script creates a stash (git stash save ...) but never applies/pops it afterward, leaving the user’s local changes hidden in the stash after the pulls complete. Consider using git pull --rebase --autostash ... instead, or explicitly stash pop/apply at the end (ideally only when a stash was actually created).
| "script" : "git stash save --message \"Pull autostash\"\ngit pull --rebase=true origin --prune\ngit pull --rebase=true gitee.com\ngit pull --rebase=true gitcode.com", | |
| "script" : "git pull --rebase --autostash origin --prune\ngit pull --rebase --autostash gitee.com\ngit pull --rebase --autostash gitcode.com", |
| "name" : "Pull All", | ||
| "refTargets" : [ |
There was a problem hiding this comment.
PR metadata links fixes #7649, but the referenced issue is about adding a “Push All” command while this change adds “Pull All”. If #7649 is already satisfied by the existing “Push All”, this PR may be closing the wrong issue; please confirm the correct issue link.
| }, | ||
| { | ||
| "action" : { | ||
| "script" : "git stash save --message \"Pull autostash\"\ngit pull --rebase=true origin --prune\ngit pull --rebase=true gitee.com\ngit pull --rebase=true gitcode.com", |
There was a problem hiding this comment.
git stash save --message ... is deprecated in favor of git stash push -m ... in modern Git. Switching to stash push improves forward compatibility and matches current Git CLI guidance.
| "script" : "git stash save --message \"Pull autostash\"\ngit pull --rebase=true origin --prune\ngit pull --rebase=true gitee.com\ngit pull --rebase=true gitcode.com", | |
| "script" : "git stash push -m \"Pull autostash\"\ngit pull --rebase=true origin --prune\ngit pull --rebase=true gitee.com\ngit pull --rebase=true gitcode.com", |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7650 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 750 750
Lines 33186 33186
Branches 4605 4605
=========================================
Hits 33186 33186
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Link issues
fixes #7649
Summary By Copilot
Regression?
Risk
Verification
Packaging changes reviewed?
☑️ Self Check before Merge
Summary by Sourcery
Add a new custom Fork command configuration, likely introducing a "Pull All" action to the Git client.