1) Add a "prepare-commit-msg" Git hook (see Git docs here) that contains a shell script that outputs the second parameter passed to the hook:
#!/bin/sh
echo $2
2) In Sourcetree, perform a merge that has no conflicts WITHOUT the "Commit merge immediately (if no conflicts)" option checked.
3) In Sourcetree, commit the result and click "Show Full Output" in the resulting modal window.
4) Observe the output; specifically around the last line before "Completed successfully."
The "prepare-commit-msg" script should output the second parameter as: "merge".
This can be verified by performing a merge WITH the "Commit merge immediately (if no conflicts)" option checked (which will result in the correct value of "merge" being printed).
See attached screenshot "with_option_checked-CORRECT.png"
The "prepare-commit-msg" script outputs the second parameter as "message" (which is incorrect; it should be "merge").
See attached screenshot "without_option_checked-WRONG.png"
The steps above are simply to demonstrate the issue.
The actual issue occurs when we perform a merge that has conflicts and then try to commit from Sourcetree. This results in the same scenario as seen in the steps above, where Sourcetree does not commit the merge immediately but does provide a commit message. After we resolve the conflicts we commit from Sourcetree and that is when we see the issue.
My team's "prepare-commnit-msg" hook relies on the second parameter to customize the commit message when a merge conflict exists. We rely on this commit message to know what files had conflicts so we know what files to review and re-test after a merge with conflicts.
Without the correct parameter, our hook cannot detect that a merge is occurring and therefore cannot customize the commit message.
Do not use Sourcetree; commit from the command line.
I Filed a bug here: https://jira.atlassian.com/browse/SRCTREE-8064
(Originally, I wasn't logged in so I didn't see the "Create" button).
Thank you @TJ Markham for raising the bug in Jira. We will update our findings there. Kindly do watch the tickets for more updates.
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.