Merge pull request #26 from barthez/skip-capital-letter-check-for-fixups-and-squashes

Skip capital letter check for autosquash commits

Author: m1foley

lib/fit_commit/validators/capitalize_subject.rb

@@ -3,8 +3,10 @@
 module FitCommit
   module Validators
     class CapitalizeSubject < Base
+      AUTOSQUASH = /\A(fixup|squash)! /
+
       def validate_line(lineno, text)
-        if lineno == 1 && text[0] =~ /[[:lower:]]/
+        if lineno == 1 && text[0] =~ /[[:lower:]]/ && text !~ AUTOSQUASH
           add_error(lineno, "Begin all subject lines with a capital letter.")
         end
       end

test/unit/validators/capitalize_subject_test.rb

@@ -17,6 +17,22 @@
       assert_empty validator.warnings
     end
   end
+  describe "subject is fixup commit" do
+    let(:commit_msg) { "fixup! foo bar" }
+    it "does not have errors/warnings" do
+      validator.validate(commit_lines)
+      assert_empty validator.errors
+      assert_empty validator.warnings
+    end
+  end
+  describe "subject is squash commit" do
+    let(:commit_msg) { "squash! foo bar" }
+    it "does not have errors/warnings" do
+      validator.validate(commit_lines)
+      assert_empty validator.errors
+      assert_empty validator.warnings
+    end
+  end
   describe "subject is capitalized" do
     let(:commit_msg) { "Foo bar" }
     it "does not have errors/warnings" do