fixes cops

Author: adamstrickland

.rubocop.yml

@@ -69,7 +69,7 @@ Style/CaseIndentation:
   Description: Indentation of when in a case/when/[else/]end.
   StyleGuide: https://github.com/bbatsov/ruby-style-guide#indent-when-to-case
   Enabled: true
-  IndentWhenRelativeTo: case
+  EnforcedStyle: case
   SupportedStyles:
   - case
   - end
@@ -374,9 +374,6 @@ Style/SpaceAroundBlockParameters:
   Description: Checks the spacing inside and after block parameters pipes.
   Enabled: true
   EnforcedStyleInsidePipes: no_space
-  SupportedStyles:
-  - space
-  - no_space
 Style/SpaceAroundEqualsInParameterDefault:
   Description: Checks that the equals signs in parameter default assignments have
     or don't have surrounding space depending on configuration.
@@ -504,6 +501,7 @@ Metrics/LineLength:
   - http
   - https
   Exclude:
+    - tmuxinator.gemspec
     - lib/tmuxinator/window.rb
     - spec/lib/tmuxinator/window_spec.rb
 Metrics/MethodLength:
@@ -525,10 +523,14 @@ Metrics/PerceivedComplexity:
   Max: 7
 Metrics/BlockLength:
   Exclude:
+    - tmuxinator.gemspec
     - lib/tmuxinator/cli.rb
     - spec/factories/**/*.rb
     - spec/matchers/**/*.rb
     - spec/**/*_spec.rb
+Lint/InverseMethods:
+  Exclude:
+    - lib/tmuxinator/project.rb
 Lint/PercentStringArray:
   Exclude:
     - lib/tmuxinator/cli.rb
@@ -543,17 +545,11 @@ Lint/AssignmentInCondition:
 Lint/EndAlignment:
   Description: Align ends correctly.
   Enabled: true
-  AlignWith: keyword
-  SupportedStyles:
-  - keyword
-  - variable
+  EnforcedStyleAlignWith: keyword
 Lint/DefEndAlignment:
   Description: Align ends corresponding to defs correctly.
   Enabled: true
-  AlignWith: start_of_line
-  SupportedStyles:
-  - start_of_line
-  - def
+  EnforcedStyleAlignWith: start_of_line
 Style/InlineComment:
   Description: Avoid inline comments.
   Enabled: false
@@ -717,7 +713,7 @@ Style/LineEndConcatenation:
   Description: Use \ instead of + or << to concatenate two string literals at line
     end.
   Enabled: false
-Style/MethodCallParentheses:
+Style/MethodCallWithoutArgsParentheses:
   Description: Do not use parentheses for method calls with no arguments.
   StyleGuide: https://github.com/bbatsov/ruby-style-guide#no-args-no-parens
   Enabled: true
@@ -876,6 +872,8 @@ Style/UnneededPercentQ:
   Description: Checks for %q/%Q when single quotes or double quotes would do.
   StyleGuide: https://github.com/bbatsov/ruby-style-guide#percent-q
   Enabled: true
+  Exclude:
+    - tmuxinator.gemspec
 # Style/UnneededPercentX:
 #   Description: Checks for %x when `` would do.
 #   StyleGuide: https://github.com/bbatsov/ruby-style-guide#percent-x

Rakefile

@@ -5,4 +5,4 @@ require "rspec/core/rake_task"
 RuboCop::RakeTask.new
 RSpec::Core::RakeTask.new
 
-task :test => ["spec", "rubocop"]
+task test: ["spec", "rubocop"]

lib/tmuxinator/project.rb

@@ -54,7 +54,7 @@ def self.load(path, options = {})
         @args = args
 
         content = Erubis::Eruby.new(raw_content).result(binding)
-        YAML.load(content)
+        YAML.safe_load(content)
       rescue SyntaxError, StandardError => error
         raise "Failed to parse config file: #{error.message}"
       end
@@ -184,7 +184,7 @@ def tmux_has_session?(name)
       # Please see issue #564.
       unescaped_name = name.shellsplit.join("")
 
-      !!(sessions =~ /^#{unescaped_name}:/)
+      !(sessions !~ /^#{unescaped_name}:/)
     end
 
     def socket

spec/factories/projects.rb

@@ -1,6 +1,7 @@
 def yaml_load(file)
-  YAML.load(File.read(File.expand_path(file)))
+  YAML.safe_load(File.read(File.expand_path(file)))
 end
+
 FactoryGirl.define do
   factory :project, class: Tmuxinator::Project do
     transient do

tmuxinator.gemspec

@@ -1,4 +1,5 @@
 # coding: utf-8
+
 lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require "tmuxinator/version"