Open
Conversation
Member
Author
|
This pull request is part of a stack:
|
Contributor
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🔴 👀 Review RequirementsWaiting for:
or
or
or
This rule is failing.
🔴 🔎 ReviewsWaiting for:
This rule is failing.
🟢 🤖 Continuous IntegrationWonderful, this rule succeeded.
🟢 Enforce conventional commitWonderful, this rule succeeded.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 📕 PR descriptionWonderful, this rule succeeded.
|
jd
force-pushed
the
devs/jd/feat/stack-reorder-move/Id3e08ba58709c88ac9eaf8fcf4137f630ba2f383
branch
from
7220a93 to
be67877
Compare
There was a problem hiding this comment.
Pull request overview
Adds a new mergify stack move command to reposition a single commit within the current stack, building on the existing non-interactive rebase helpers introduced for stack reordering.
Changes:
- Introduce
mergify_cli.stack.move.stack_move()to compute a new commit order and execute (or preview) the rebase. - Wire the new
movesubcommand into the stack CLI and document usage in the stack skill guide. - Add end-to-end style tests for move scenarios (before/after/first/last, dry-run, and validation cases).
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
mergify_cli/stack/move.py |
New implementation of commit movement using existing reorder/rebase helpers. |
mergify_cli/stack/cli.py |
Registers the mergify stack move command and its arguments/options. |
mergify_cli/stack/skill.md |
Documents the new move syntax alongside reorder guidance and examples. |
mergify_cli/tests/stack/test_move.py |
Adds functional tests covering move behavior and error cases. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Base automatically changed from
devs/jd/feat/stack-reorder-move/I46dc35fd28e3535e9f64f85581db87a50b6d4c21
to
main
April 10, 2026 14:09
Contributor
|
@jd this pull request is now in conflict 😩 |
jd
force-pushed
the
devs/jd/feat/stack-reorder-move/Id3e08ba58709c88ac9eaf8fcf4137f630ba2f383
branch
from
be67877 to
689a87d
Compare
JulianMaurin
previously approved these changes
Apr 10, 2026
Contributor
JulianMaurin
left a comment
JulianMaurin
left a comment
There was a problem hiding this comment.
not sure about this one 🤔
kozlek
previously approved these changes
Apr 10, 2026
Contributor
|
@jd this pull request is now in conflict 😩 |
Add `mergify stack move` to move a single commit within the stack. Supports natural positional syntax: mergify stack move <commit> before <target> mergify stack move <commit> after <target> mergify stack move <commit> first mergify stack move <commit> last Reuses the shared rebase helpers from the reorder module. Supports --dry-run and both SHA and Change-Id prefix identification. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Change-Id: Id3e08ba58709c88ac9eaf8fcf4137f630ba2f383 Claude-Session-Id: cdc0d010-0b6d-4808-a13c-d8e205472a4b
jd
force-pushed
the
devs/jd/feat/stack-reorder-move/Id3e08ba58709c88ac9eaf8fcf4137f630ba2f383
branch
from
689a87d to
af55a58
Compare
mergify
bot
dismissed stale reviews from kozlek and JulianMaurin
Pull request has been modified.
JulianMaurin
approved these changes
Apr 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add
mergify stack moveto move a single commit within the stack.Supports natural positional syntax:
mergify stack move before
mergify stack move after
mergify stack move first
mergify stack move last
Reuses the shared rebase helpers from the reorder module.
Supports --dry-run and both SHA and Change-Id prefix identification.
Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com
Claude-Session-Id: cdc0d010-0b6d-4808-a13c-d8e205472a4b