Skip to content

Commit 13e5fc1

Browse files
fix: Fix IsValidMessage method adding a new exception for merge from master to the branch
1 parent 247ac03 commit 13e5fc1

File tree

3 files changed

+9
-1
lines changed

3 files changed

+9
-1
lines changed

CHANGELOG.md

+3
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,6 @@
1+
v2.0.3:
2+
- Fix IsValidMessage method adding a new exception for merge from master to the branch (@esequiel.virtuoso)
3+
14
v2.0.2:
25
- Fix IsValidMessage method adding a new exception for merge from master to the branch (@esequiel.virtuoso)
36

src/commit-message/commit_message_manager.go

+2-1
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,8 @@ func isMergeMasterToBranch(message string) bool {
7979
splitedMessage := strings.Split(strings.ToLower(message), "\n")
8080

8181
for _, row := range splitedMessage {
82-
if strings.Contains(row, "'origin/master' into") || strings.Contains(row, "merge branch 'master' into") {
82+
lowerRow := strings.ToLower(row)
83+
if strings.Contains(lowerRow, "'origin/master' into") || strings.Contains(lowerRow, "merge branch 'master' into") || strings.Contains(lowerRow, "merge branch 'master' of") {
8384
return true
8485
}
8586
}

src/commit-message/commit_message_manager_test.go

+4
Original file line numberDiff line numberDiff line change
@@ -104,4 +104,8 @@ func TestIsValidMessageMergeMasterBranchSuccess(t *testing.T) {
104104
message = "first message row \n Merge branch 'master' into something \n last message row"
105105
actual = f.commitMessageManager.IsValidMessage(message)
106106
tests.AssertTrue(t, actual)
107+
108+
message = "first message row \n Merge branch 'master' of something \n last message row"
109+
actual = f.commitMessageManager.IsValidMessage(message)
110+
tests.AssertTrue(t, actual)
107111
}

0 commit comments

Comments
 (0)