Fix an issue reported by rubocop that wasn't ignored in the todo config

Issue text: Favor modifier if/unless usage when you have a single-line body.

lib/homesick.rb

@@ -31,9 +31,7 @@ class Homesick < Thor
       destination = nil
       if File.exist?(uri)
         uri = Pathname.new(uri).expand_path
-        if uri.to_s.start_with?(repos_dir.to_s)
+        raise "Castle already cloned to #{uri}" if uri.to_s.start_with?(repos_dir.to_s)
-          raise "Castle already cloned to #{uri}"
-        end
 
         destination = uri.basename
 

lib/homesick/actions.rb

@@ -93,9 +93,7 @@ class Homesick
       if destination.exist?
         say_status :conflict, "#{destination} exists", :red unless options[:quiet]
 
-        if options[:force] || shell.file_collision(destination) { source }
+        system "mv '#{source}' '#{destination}'" if (options[:force] || shell.file_collision(destination) { source }) && !options[:pretend]
-          system "mv '#{source}' '#{destination}'" unless options[:pretend]
-        end
       else
         # this needs some sort of message here.
         system "mv '#{source}' '#{destination}'" unless options[:pretend]
@@ -155,9 +153,7 @@ class Homesick
         else
           say_status :conflict, "#{destination} exists and points to #{destination.readlink}", :red unless options[:quiet]
 
-          if options[:force] || shell.file_collision(destination) { source }
+          system "ln -nsf '#{source}' '#{destination}'" if (options[:force] || shell.file_collision(destination) { source }) && !options[:pretend]
-            system "ln -nsf '#{source}' '#{destination}'" unless options[:pretend]
-          end
         end
       elsif destination.exist?
         say_status :conflict, "#{destination} exists", :red unless options[:quiet]

rubocop-todo.yml

@@ -5,11 +5,8 @@
 BlockAlignment:
   Enabled: false
 
-BlockNesting:
-  Max: 4
-
 ClassLength:
-  Max: 352
+  Max: 350
 
 CommentAnnotation:
   Enabled: false