-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for "--rebase-merges" for newest version of git #6920
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6920 +/- ##
==========================================
- Coverage 47.69% 47.64% -0.06%
==========================================
Files 736 736
Lines 54054 54056 +2
Branches 7086 7103 +17
==========================================
- Hits 25782 25754 -28
- Misses 26877 26883 +6
- Partials 1395 1419 +24
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Few minor comments
@@ -605,7 +605,7 @@ public void RebaseCmd() | |||
"-c rebase.autoSquash=false rebase -i --no-autosquash \"branch\"", | |||
GitCommandHelpers.RebaseCmd("branch", interactive: true, preserveMerges: false, autosquash: false, autoStash: false).Arguments); | |||
Assert.AreEqual( | |||
"-c rebase.autoSquash=false rebase --preserve-merges \"branch\"", | |||
"-c rebase.autoSquash=false rebase --rebase-merges \"branch\"", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests should pivot on git version.
So if someone runs a codebase against an older version of git (like me while testing) they shouldn't get a test failure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: We have a few tests for features not in the oldest supported version, but those tests do not use Git binaries (only argument tests).
@@ -617,7 +617,7 @@ public void RebaseCmd() | |||
"-c rebase.autoSquash=false rebase -i --autosquash \"branch\"", | |||
GitCommandHelpers.RebaseCmd("branch", interactive: true, preserveMerges: false, autosquash: true, autoStash: false).Arguments); | |||
Assert.AreEqual( | |||
"-c rebase.autoSquash=false rebase -i --autosquash --preserve-merges --autostash \"branch\"", | |||
"-c rebase.autoSquash=false rebase -i --autosquash --rebase-merges --autostash \"branch\"", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
appveyor.yml
Outdated
@@ -1,6 +1,6 @@ | |||
version: 3.2.0.{build} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's submit this change separately, as it is orthogonal to the main change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that was a last test (before going to bed ;) ) to gather information, not planned to be merged...
47a75fa
to
2df922e
Compare
now that we recommend v2.22 (gitextensions#6769) that deprecate "--preserve-merges" See: * for option deprection: git/git@fa1b86e#diff-c7361e406139e8cd3a300b80b8f8cc8dR1220 * for some doc: https://stackoverflow.com/questions/15915430/what-exactly-does-gits-rebase-preserve-merges-do-and-why/50555740#50555740
2df922e
to
519c0dc
Compare
done. |
We could include this in 3.2 |
Thank you |
now that we recommend v2.22 (#6769) that deprecate "--preserve-merges"
See:
Test methodology
Test environment(s)