ApprovalTests.Java icon indicating copy to clipboard operation
ApprovalTests.Java copied to clipboard

Switch arguments to diff-tool

Open roesslerj opened this issue 8 years ago • 2 comments

Currently, the diff-tool is called with arguments: .received. .approved., causing the expected result to be on the right side of the merge/diff-tool. Default is that the expected result is to the left.

Could the order of the arguments be switched, so the approved result shows up on the left?

Great! Thanks!

roesslerj avatar Jun 28 '16 18:06 roesslerj

No. Many of the diff tools will not let you move changes from right to left. Since the approved file is often created in the diff tool by moving the results you got to the approved side reversing the order is problematic.

Llewellyn Falco www.approvaltests.com

On Tue, Jun 28, 2016 at 2:46 PM, Jeremias Rößler [email protected] wrote:

Currently, the diff-tool is called with arguments: .received. .approved., causing the expected result to be on the right side of the merge/diff-tool. Default is that the expected result is to the left.

Could the order of the arguments be switched, so the approved result shows up on the left?

Great! Thanks!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/approvals/ApprovalTests.Java/issues/26, or mute the thread https://github.com/notifications/unsubscribe/AAAqehO3qQh1u8ddvVXc6VcaghYVZRNPks5qQWvrgaJpZM4JAaQR .

isidore avatar Jun 28 '16 19:06 isidore

Thanks for the fast feedback!

roesslerj avatar Jun 28 '16 21:06 roesslerj