Merge pull request #108 from codacy/ls/change-regex-expression

fix: only consider major when commit message starts with `breaking:`

Author: CallMeLeopoldo

Makefile

@@ -1,7 +1,6 @@
 CRYSTAL?=$(shell which crystal)
 CRYSTAL_FLAGS=--release
 CRYSTAL_STATIC_FLAGS=--static
-
 VERSION?=$(shell cat .version)
 
 all: fmt test build docker_build ## clean and produce target binary and docker image

spec/git-version-spec.cr

@@ -474,7 +474,7 @@ describe GitVersion do
       tmp.exec %(git checkout -b master)
       tmp.exec %(git commit --no-gpg-sign --allow-empty -m "1")
       tmp.exec %(git tag "0.1.0")
-      tmp.exec %(git commit --no-gpg-sign --allow-empty -m ":breaking: 2")
+      tmp.exec %(git commit --no-gpg-sign --allow-empty -m "breaking: 2")
 
       version = git.get_new_version
       version.should eq("1.0.0")
@@ -495,7 +495,7 @@ describe GitVersion do
       tmp.exec %(git tag "0.1.0")
       tmp.exec %(git commit --no-gpg-sign --allow-empty -m "feature: 2")
       tmp.exec %(git tag "0.2.0-asd")
-      tmp.exec %(git commit --no-gpg-sign --allow-empty -m ":breaking: 2")
+      tmp.exec %(git commit --no-gpg-sign --allow-empty -m "breaking: 2")
 
       version = git.get_new_version
       version.should eq("1.0.0")

src/git-version.cr

@@ -152,11 +152,7 @@ module GitVersion
       major = false
       get_commits_since(previous_tag).each do |c|
         commit = c.downcase
-        match = if @major_id_is_regex
-          /#{@major_identifier}/.match(commit)
-        else
-          commit.includes?(@major_identifier)
-        end
+        match = /^#{@major_identifier}/.match(commit)
         if match
           previous_version =
             SemanticVersion.new(