Fix coding standards

lib/producer/core/cli.rb

@@ -38,7 +38,8 @@ module Producer
 
       attr_reader :arguments, :stdin, :stdout, :stderr, :env
 
-      def initialize(args, environment, stdin: $stdin, stdout: $stdout, stderr: $stderr)
+      def initialize(args, environment, stdin: $stdin, stdout: $stdout,
+          stderr: $stderr)
         @arguments  = args
         @stdin      = stdin
         @stdout     = stdout
@@ -82,7 +83,7 @@ module Producer
       def split_arguments_lists(arguments)
         arguments
           .chunk  { |e| e == ARGUMENTS_SEPARATOR }
-          .reject { |b, a| b }
+          .reject { |a, _| a }
           .map    &:last
       end
 

lib/producer/core/env.rb

@@ -7,7 +7,8 @@ module Producer
       attr_reader   :input, :output, :error_output, :registry, :logger
       attr_accessor :target, :verbose, :debug, :dry_run, :recipe_argv
 
-      def initialize(input: $stdin, output: $stdout, error_output: $stderr, remote: nil, registry: {})
+      def initialize(input: $stdin, output: $stdout, error_output: $stderr,
+          remote: nil, registry: {})
         @verbose = @debug = @dry_run = false
         @input        = input
         @output       = output

lib/producer/core/error_formatter.rb

@@ -56,7 +56,7 @@ module Producer
       end
 
       def indent_backtrace(backtrace)
-        backtrace.map { |e| '  %s' % e }
+        backtrace.map { |e| '  ' + e }
       end
     end
   end

lib/producer/core/logger_formatter.rb

@@ -1,7 +1,7 @@
 module Producer
   module Core
     class LoggerFormatter < Logger::Formatter
-      def call(severity, datetime, progname, message)
+      def call(severity, _datetime, _progname, message)
         prefix(severity) + message + "\n"
       end
 

lib/producer/core/recipe.rb

@@ -46,7 +46,7 @@ module Producer
       end
 
       def compose_macro(name, macro, *base_args)
-        self.class.class_eval { compose_macro name, macro, *base_args}
+        self.class.class_eval { compose_macro name, macro, *base_args }
       end
 
       def test_macro(name, &block)

lib/producer/core/remote.rb

@@ -26,16 +26,16 @@ module Producer
 
       def execute(command, output = '', error_output = '')
         session.open_channel do |channel|
-          channel.exec command do |ch, success|
+          channel.exec command do |ch, _success|
-            ch.on_data do |c, data|
+            ch.on_data do |_c, data|
               output << data
             end
 
-            ch.on_extended_data do |c, type, data|
+            ch.on_extended_data do |_c, _type, data|
               error_output << data
             end
 
-            ch.on_request 'exit-status' do |c, data|
+            ch.on_request 'exit-status' do |_c, data|
               exit_status = data.read_long
               fail RemoteCommandExecutionError, command if exit_status != 0
             end

lib/producer/core/remote/fs.rb

@@ -29,11 +29,11 @@ module Producer
         end
 
         def mkdir(path, attributes = {})
-          ret = sftp.mkdir! path, attributes
+          sftp.mkdir! path, attributes
         end
 
         def file_read(path)
-          sftp.file.open(path) { |f| content = f.read }
+          sftp.file.open(path) { |f| f.read }
         rescue Net::SFTP::StatusException
           nil
         end

lib/producer/core/task.rb

@@ -52,13 +52,20 @@ module Producer
         @actions << self.class.evaluate(@env, name, *args, &block)
       end
 
-      def ask(question, choices, prompter: Prompter.new(@env.input, @env.output))
+      def ask(question, choices, prompter: build_prompter)
         prompter.prompt(question, choices)
       end
 
       def template(path, variables = {})
         Template.new(path).render variables
       end
+
+
+      private
+
+      def build_prompter
+        Prompter.new(@env.input, @env.output)
+      end
     end
   end
 end