Merge pull request #8 from shopsmart/enforce_linter

Enforce Linter

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    bd_lint (0.2.1.4)
+    bd_lint (0.2.2)
       brakeman
       bundler-audit
       execjs
@@ -206,4 +206,4 @@ DEPENDENCIES
   thor
 
 BUNDLED WITH
-   1.16.0
+   1.16.1

lib/bd_lint.rb

@@ -1,10 +1,14 @@
 require "bd_lint/rvm_version"
+require "bd_lint/validator"
 require "pre-commit"
 require "plugins/pre_commit/checks/jscs"
 
+# Run this on application initialization
+BdLint::Validator.check
+BdLint::RvmVersion.check
+
 module BdLint
   def self.run
-    RvmVersion.check
     PreCommit.run
   end
 end

lib/bd_lint/rvm_version.rb

@@ -1,7 +1,7 @@
+require "mkmf"
+
 module BdLint
   class RvmVersion
-    require "mkmf"
-
     def self.check
       if MakeMakefile.find_executable("rvm")
         rvm_current = %x(rvm current).chomp

lib/bd_lint/validator.rb

@@ -0,0 +1,19 @@
+module BdLint
+  class Validator
+    GIT_HOOK_PATH = ".git/hooks/pre-commit".freeze
+
+    class GitHookError < StandardError; end
+
+    def self.check
+      raise GitHookError, "Please run `bundle exec #{message}`" if invalid?
+    end
+
+    def self.invalid?
+      !defined?(Rake) && !File.exist?(GIT_HOOK_PATH)
+    end
+
+    def self.message
+      defined?(Rails) ? "rails g pre_commit" : "rake bd_lint:generate:pre_commit"
+    end
+  end
+end

lib/bd_lint/version.rb

@@ -1,3 +1,3 @@
 module BdLint
-  VERSION = "0.2.1.4".freeze
+  VERSION = "0.2.2".freeze
 end

test_app/config/application.rb

@@ -6,6 +6,8 @@
 require "action_mailer/railtie"
 require "action_view/railtie"
 require "sprockets/railtie"
+# Prevents test from not running
+require "rake"
 # require "rails/test_unit/railtie"
 
 Bundler.require(*Rails.groups)