Upgrade RuboCop 0.39.0 -> 0.40.0 for Travis

Author: sds

Gemfile

@@ -12,4 +12,4 @@ gem 'travis', '~> 1.7'
 gem 'coveralls'
 
 # Pin RuboCop for Travis builds.
-gem 'rubocop', '0.39.0'
+gem 'rubocop', '0.40.0'

lib/overcommit/utils.rb

@@ -220,7 +220,8 @@ def processor_count # rubocop:disable all
             if Overcommit::OS.windows?
               require 'win32ole'
               result = WIN32OLE.connect('winmgmts://').ExecQuery(
-                'select NumberOfLogicalProcessors from Win32_Processor')
+                'select NumberOfLogicalProcessors from Win32_Processor'
+              )
               result.to_enum.collect(&:NumberOfLogicalProcessors).reduce(:+)
             elsif File.readable?('/proc/cpuinfo')
               IO.read('/proc/cpuinfo').scan(/^processor/).size
@@ -284,9 +285,10 @@ def convert_glob_to_absolute(glob)
       # @param pattern [String]
       # @param path [String]
       def matches_path?(pattern, path)
-        File.fnmatch?(pattern, path,
-                      File::FNM_PATHNAME | # Wildcard doesn't match separator
-                      File::FNM_DOTMATCH   # Wildcards match dotfiles
+        File.fnmatch?(
+          pattern, path,
+          File::FNM_PATHNAME | # Wildcard doesn't match separator
+          File::FNM_DOTMATCH   # Wildcards match dotfiles
         )
       end
 

spec/overcommit/hook/pre_commit/forbidden_branches_spec.rb

@@ -4,12 +4,15 @@
   let(:default_config) { Overcommit::ConfigurationLoader.default_configuration }
   let(:branch_patterns) { ['master', 'release/*'] }
   let(:config) do
-    default_config.merge(Overcommit::Configuration.new(
-      'PreCommit' => {
-        'ForbiddenBranches' => {
-          'branch_patterns' => branch_patterns
+    default_config.merge(
+      Overcommit::Configuration.new(
+        'PreCommit' => {
+          'ForbiddenBranches' => {
+            'branch_patterns' => branch_patterns
+          }
         }
-      }))
+      )
+    )
   end
   let(:context) { double('context') }
   subject { described_class.new(config, context) }

spec/overcommit/hook/pre_commit/go_lint_spec.rb

@@ -27,8 +27,9 @@
     context 'and it reports an error' do
       context 'on stdout' do
         before do
-          result.stub(stderr: '', stdout:
-            'file1.go:1:1: error should be the last type when returning multiple items'
+          result.stub(
+            stderr: '',
+            stdout: 'file1.go:1:1: error should be the last type when returning multiple items',
           )
         end
 
@@ -37,8 +38,9 @@
 
       context 'on stderr' do
         before do
-          result.stub(stdout: '', stderr:
-            "file1.go:1:1: expected 'package', found 'IDENT' foo"
+          result.stub(
+            stdout: '',
+            stderr: "file1.go:1:1: expected 'package', found 'IDENT' foo"
           )
         end
 

spec/overcommit/hook/pre_commit/go_vet_spec.rb

@@ -29,8 +29,8 @@
 
     context 'when go tool vet is not installed' do
       before do
-        result.stub(stderr:
-          'go tool: no such tool "vet"; to install:'
+        result.stub(
+          stderr: 'go tool: no such tool "vet"; to install:',
         )
       end
 
@@ -39,8 +39,8 @@
 
     context 'and it reports an error' do
       before do
-        result.stub(stderr:
-          'file1.go:1: possible formatting directive in Print call'
+        result.stub(
+          stderr: 'file1.go:1: possible formatting directive in Print call',
         )
       end