Remove exclusion of id prefix

rockstaedt · Aug 4, 2023 · 1bab9e4 · 1bab9e4
1 parent 146f6f1
commit 1bab9e4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/internal/model/commit_message.go b/internal/model/commit_message.go
@@ -1,7 +1,6 @@
 package model
 
 import (
-	"regexp"
 	"strings"
 )
 
@@ -26,15 +25,8 @@ func (cm *CommitMessage) ValidateSubject() int {
 		return 0
 	}
 
-	if currentSubjectLength > 72 {
-		return currentSubjectLength - 50
-	}
-
 	if currentSubjectLength > 50 {
-		re := regexp.MustCompile(`^#\d+ -\s*(.*)$`)
-		trimmedSubject := re.ReplaceAllString(cm.Subject, `$1`)
-
-		return len(trimmedSubject) - 50
+		return currentSubjectLength - 50
 	}
 
 	return 0

diff --git a/internal/model/commit_message_test.go b/internal/model/commit_message_test.go
@@ -62,7 +62,7 @@ func TestNewCommitMessage(t *testing.T) {
 		}{
 			{"more than................72....................................characters", 23},
 			{"more than................50................less than 72 characters", 16},
-			{"#1301 - more than........50..............through ID prefix", 0},
+			{"#1301 - more than........50..............through ID prefix", 8},
 			{"short subject line", 0},
 			{"Merge pull request commits are ignored because they can easily exceed 52 characters", 0},
 			{"Merge branch commits are also ignored..............................................", 0},