RubyGems - overcommit - Versions diffs - 0.46.0 → 0.47.0 - Mend

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Checks the format of an author's email address.
   class AuthorEmail < Base

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Ensures that a commit author has a name with at least first and last names.
   class AuthorName < Base

data/lib/overcommit/hook/pre_commit/base.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require 'forwardable'
 require 'overcommit/utils/messages_utils'

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Checks for broken symlinks.
   class BrokenSymlinks < Base

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Checks for vulnerable versions of gems in Gemfile.lock.
   #

@@ -22,7 +22,8 @@ module Overcommit::Hook::PreCommit
       new_lockfile = File.read(LOCK_FILE) if File.exist?(LOCK_FILE)
       if previous_lockfile != new_lockfile
-        return :fail, "#{LOCK_FILE} is not up-to-date -- run `#{command.join(' ')}`"
+        return :fail, "#{LOCK_FILE} is not up-to-date -- run \
+        `#{command.join(' ')}` or add the Gemfile and/or Gemfile.lock".squeeze
       end
       :pass

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Check if any gems in Gemfile.lock have newer versions, unless the
   # Gemfile.lock is ignored by Git.

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Checks for files that would conflict in case-insensitive filesystems
   # Adapted from https://github.com/pre-commit/pre-commit-hooks

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `chamber compare` against a configurable set of namespaces.
   #

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `chamber secure` against any modified Chamber settings files.
   #

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `chamber sign --verify`.
   #
@@ -5,7 +7,7 @@ module Overcommit::Hook::PreCommit
   # rubocop:disable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
   class ChamberVerification < Base
     def run
-      approver_name  = config.fetch('approver_name', 'your approver')
+      approver_name  = config.fetch('approver_name') { 'your approver' }
       approver_email = config['approver_email'] ? " (#{config['approver_email']})" : nil
       result = execute(command)

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `coffeelint` against any modified CoffeeScript files.
   #

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+module Overcommit::Hook::PreCommit
+  # Runs `cookstyle` against any modified Chef Ruby files.
+  #
+  # @see https://docs.chef.io/cookstyle.html
+  class CookStyle < Base
+    GENERIC_MESSAGE_TYPE_CATEGORIZER = lambda do |type|
+      type =~ /^warn/ ? :warning : :error
+    end
+    COP_MESSAGE_TYPE_CATEGORIZER = lambda do |type|
+      type.include?('W') ? :warning : :error
+    end
+    def run
+      result = execute(command, args: applicable_files)
+      return :pass if result.success?
+      generic_messages = extract_messages(
+        result.stderr.split("\n"),
+        /^(?<type>[a-z]+)/i,
+        GENERIC_MESSAGE_TYPE_CATEGORIZER,
+      )
+      cop_messages = extract_messages(
+        result.stdout.split("\n"),
+        /^(?<file>(?:\w:)?[^:]+):(?<line>\d+):[^ ]+ (?<type>[^ ]+)/,
+        COP_MESSAGE_TYPE_CATEGORIZER,
+      )
+      generic_messages + cop_messages
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/credo.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `credo` against any modified ex files.
   #

data/lib/overcommit/hook/pre_commit/css_lint.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `csslint` against any modified CSS files.
   #

data/lib/overcommit/hook/pre_commit/dogma.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `dogma` against any modified ex files.
   #

data/lib/overcommit/hook/pre_commit/es_lint.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `eslint` against any modified JavaScript files.
   #

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Checks for files with execute permissions, which are usually not necessary
   # in source code files (and are typically caused by a misconfigured editor

data/lib/overcommit/hook/pre_commit/fasterer.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `fasterer` against any modified Ruby files.
   #

data/lib/overcommit/hook/pre_commit/fix_me.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Check for "token" strings
   class FixMe < Base

data/lib/overcommit/hook/pre_commit/flay.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `flay` against any modified files.
   #

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `foodcritic` against any modified Ruby files from Chef directory structure.
   #

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Prevents commits to branches matching one of the configured patterns.
   class ForbiddenBranches < Base

data/lib/overcommit/hook/pre_commit/go_lint.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `golint` against any modified Golang files.
   #

data/lib/overcommit/hook/pre_commit/go_vet.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `go vet` against any modified Golang files.
   #

data/lib/overcommit/hook/pre_commit/hadolint.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 module Overcommit::Hook::PreCommit
   # Runs `hadolint` against any modified Dockefile files.
   #