stack_master 2.17.1 → 2.18.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e73f1b3d79a230203a083898d14d256ac7c61fb0dd4a49627af097759341ca96
-  data.tar.gz: 193c5824b8972d6ddc36f66dd23d83360c1af43562a3b038c413bb6ff624626a
+  metadata.gz: ffa37da4e3a6d6d80804a60be1b868180a67470009472b02a0c838354028efb6
+  data.tar.gz: e04ddf265d427425f0a1a60973eec74e2e6cc42e3cc47f00b8cbf584f1cdac93
 SHA512:
-  metadata.gz: 41d00dda80224d66ae46e73746b43f1e0738ef075ad8ce3122494db579c545b6cd2b15497622cf5aa683efa2bf6f67f411a031c1cb5063c0a94b40dcef8261a0
-  data.tar.gz: e26a3fa7eeae1b2be191fa579e585f34f8f781c4b94e68d42c0619a3d1effbe86cd0761b119a79058a86d98f113d6db573dfbfd0b733792ca9d21554de98684c
+  metadata.gz: fb887f7ebe035ec7ffd808af93ad5cd69329a161dfcef8f6986e1eb79d975ce81c8b7d344403d21ca9bcaa3faafa9e91d1ef50b17cbdf79d35f88fd4cf69926f
+  data.tar.gz: d59f604ca4b6881c928525ea5c8c88f3d07a8dde8899b27ed427223f6102d78c27a59e1cbcb51588e39653221c3c90fceaa758b809ab2d161df8d6bfed3da036

data/README.md

@@ -2,7 +2,7 @@
 
 [![License MIT](https://img.shields.io/badge/license-MIT-brightgreen.svg)](https://github.com/envato/stack_master/blob/master/LICENSE.md)
 [![Gem Version](https://badge.fury.io/rb/stack_master.svg)](https://badge.fury.io/rb/stack_master)
-[![Build Status](https://github.com/envato/stack_master/workflows/tests/badge.svg?branch=master)](https://github.com/envato/stack_master/actions?query=workflow%3Atests+branch%3Amaster)
+[![Build Status](https://github.com/envato/stack_master/actions/workflows/test.yml/badge.svg)](https://github.com/envato/stack_master/actions/workflows/test.yml)
 
 StackMaster is a CLI tool to manage [CloudFormation](https://aws.amazon.com/cloudformation/) stacks, with the following features:
 

data/bin/stack_master

@@ -2,14 +2,12 @@
 
 require 'stack_master'
 
-if ENV['STUB_AWS'] == 'true'
-  require 'stack_master/testing'
-end
+require 'stack_master/testing' if ENV['STUB_AWS'] == 'true'
 
-trap("SIGINT") { raise StackMaster::CtrlC }
+trap('SIGINT') { raise StackMaster::CtrlC }
 
 begin
   StackMaster::CLI.new(ARGV.dup).execute!
 rescue StackMaster::CtrlC
-  StackMaster.stdout.puts "Exiting..."
+  StackMaster.stdout.puts 'Exiting...'
 end

data/lib/stack_master/aws_driver/cloud_formation.rb

@@ -8,10 +8,10 @@ module StackMaster
       end
 
       def set_region(value)
-        if region != value
-          @region = value
-          @cf = nil
-        end
+        return if region == value
+
+        @region = value
+        @cf = nil
       end
 
       def_delegators(

data/lib/stack_master/aws_driver/s3.rb

@@ -24,11 +24,11 @@ module StackMaster
             prefix: prefix,
             bucket: bucket
           }
-        ).map(&:contents).flatten.inject({}) { |h, obj|
+        ).map(&:contents).flatten.inject({}) do |h, obj|
           h.merge(obj.key => obj)
-        }
+        end
 
-        StackMaster.stdout.puts "Uploading files to S3:"
+        StackMaster.stdout.puts 'Uploading files to S3:'
 
         files.each do |template, file|
           body = file.fetch(:body)
@@ -36,7 +36,7 @@ module StackMaster
           object_key = template.dup
           object_key.prepend("#{prefix}/") if prefix
           compiled_template_md5 = Digest::MD5.hexdigest(body).to_s
-          s3_md5 = current_objects[object_key] ? current_objects[object_key].etag.gsub("\"", '') : nil
+          s3_md5 = current_objects[object_key] ? current_objects[object_key].etag.gsub('"', '') : nil
 
           next if compiled_template_md5 == s3_md5
 
@@ -51,14 +51,14 @@ module StackMaster
               metadata: { md5: compiled_template_md5 }
             }
           )
-          StackMaster.stdout.puts "done."
+          StackMaster.stdout.puts 'done.'
         end
       end
 
       def url(bucket:, prefix:, region:, template:)
         if region == 'us-east-1'
-          ["https://s3.amazonaws.com", bucket, prefix, template].compact.join('/')
-        elsif region.start_with? "cn-"
+          ['https://s3.amazonaws.com', bucket, prefix, template].compact.join('/')
+        elsif region.start_with? 'cn-'
           ["https://s3.#{region}.amazonaws.com.cn", bucket, prefix, template].compact.join('/')
         else
           ["https://s3-#{region}.amazonaws.com", bucket, prefix, template].compact.join('/')

data/lib/stack_master/change_set.rb

@@ -1,9 +1,9 @@
 module StackMaster
   class ChangeSet
-    END_STATES = [
-      'CREATE_COMPLETE',
-      'DELETE_COMPLETE',
-      'FAILED'
+    END_STATES = %w[
+      CREATE_COMPLETE
+      DELETE_COMPLETE
+      FAILED
     ]
 
     def self.generate_change_set_name(stack_name)
@@ -50,7 +50,7 @@ module StackMaster
       @response.changes.each do |change|
         display_resource_change(io, change.resource_change)
       end
-      io.puts "========================================"
+      io.puts '========================================'
     end
 
     def failed?
@@ -84,13 +84,9 @@ module StackMaster
     def display_resource_change_detail(io, action_name, color, detail)
       target_name = [detail.target.attribute, detail.target.name].compact.join('.')
       detail_messages = [target_name]
-      if action_name == 'Replace'
-        detail_messages << "#{detail.target.requires_recreation} requires recreation"
-      end
+      detail_messages << "#{detail.target.requires_recreation} requires recreation" if action_name == 'Replace'
       triggered_by = [detail.change_source, detail.causing_entity].compact.join('.')
-      if detail.evaluation != 'Static'
-        triggered_by << "(#{detail.evaluation})"
-      end
+      triggered_by << "(#{detail.evaluation})" if detail.evaluation != 'Static'
       detail_messages << "Triggered by: #{triggered_by}"
       io.puts Rainbow("- #{detail_messages.join('. ')}. ").color(color)
     end

data/lib/stack_master/cli.rb

@@ -6,7 +6,11 @@ module StackMaster
     include Commander::Methods
 
     def initialize(argv, stdin = STDIN, stdout = STDOUT, stderr = STDERR, kernel = Kernel)
-      @argv, @stdin, @stdout, @stderr, @kernel = argv, stdin, stdout, stderr, kernel
+      @argv = argv
+      @stdin = stdin
+      @stdout = stdout
+      @stderr = stderr
+      @kernel = kernel
       Commander::Runner.instance_variable_set('@instance', Commander::Runner.new(argv))
       StackMaster.stdout = @stdout
       StackMaster.stderr = @stderr
@@ -49,7 +53,7 @@ module StackMaster
                  'Valid Values: [ DO_NOTHING | ROLLBACK | DELETE ]. ' \
                  "Default: ROLLBACK\n" \
                  'Note: You cannot use this option with Serverless Application Model (SAM) templates.'
-        c.option '--yes-param PARAM_NAME', String, "Auto-approve stack updates when only parameter PARAM_NAME changes"
+        c.option '--yes-param PARAM_NAME', String, 'Auto-approve stack updates when only parameter PARAM_NAME changes'
         c.action do |args, options|
           options.default config: default_config_file
           execute_stacks_command(StackMaster::Commands::Apply, args, options)
@@ -59,7 +63,7 @@ module StackMaster
       command :outputs do |c|
         c.syntax = 'stack_master outputs [region_or_alias] [stack_name]'
         c.summary = 'Displays outputs for a stack'
-        c.description = "Displays outputs for a stack"
+        c.description = 'Displays outputs for a stack'
         c.action do |args, options|
           options.default config: default_config_file
           execute_stacks_command(StackMaster::Commands::Outputs, args, options)
@@ -73,10 +77,10 @@ module StackMaster
         c.option('--overwrite', 'Overwrite existing files')
         c.action do |args, options|
           options.default config: default_config_file
-          unless args.size == 2
-            say "Invalid arguments. stack_master init [region] [stack_name]"
-          else
+          if args.size == 2
             StackMaster::Commands::Init.perform(options, *args)
+          else
+            say 'Invalid arguments. stack_master init [region] [stack_name]'
           end
         end
       end
@@ -94,8 +98,8 @@ module StackMaster
 
       command :events do |c|
         c.syntax = 'stack_master events [region_or_alias] [stack_name]'
-        c.summary = "Shows events for a stack"
-        c.description = "Shows events for a stack"
+        c.summary = 'Shows events for a stack'
+        c.description = 'Shows events for a stack'
         c.example 'show events for myapp-vpc in us-east-1', 'stack_master events us-east-1 myapp-vpc'
         c.option '--number Integer', Integer, 'Number of recent events to show'
         c.option '--all', 'Show all events'
@@ -108,8 +112,8 @@ module StackMaster
 
       command :resources do |c|
         c.syntax = 'stack_master resources [region] [stack_name]'
-        c.summary = "Shows stack resources"
-        c.description = "Shows stack resources"
+        c.summary = 'Shows stack resources'
+        c.description = 'Shows stack resources'
         c.action do |args, options|
           options.default config: default_config_file
           execute_stacks_command(StackMaster::Commands::Resources, args, options)
@@ -122,7 +126,7 @@ module StackMaster
         c.description = 'List stack definitions'
         c.action do |args, options|
           options.default config: default_config_file
-          say "Invalid arguments." if args.size > 0
+          say 'Invalid arguments.' if args.size > 0
           config = load_config(options.config)
           StackMaster::Commands::ListStacks.perform(config, nil, options)
         end
@@ -142,8 +146,8 @@ module StackMaster
 
       command :lint do |c|
         c.syntax = 'stack_master lint [region_or_alias] [stack_name]'
-        c.summary = "Check the stack definition locally"
-        c.description = "Runs cfn-lint on the template which would be sent to AWS on apply"
+        c.summary = 'Check the stack definition locally'
+        c.description = 'Runs cfn-lint on the template which would be sent to AWS on apply'
         c.example 'run cfn-lint on stack myapp-vpc with us-east-1 settings', 'stack_master lint us-east-1 myapp-vpc'
         c.action do |args, options|
           options.default config: default_config_file
@@ -154,7 +158,7 @@ module StackMaster
       command :nag do |c|
         c.syntax = 'stack_master nag [region_or_alias] [stack_name]'
         c.summary = "Check this stack's template with cfn_nag"
-        c.description = "Runs SAST scan cfn_nag on the template"
+        c.description = 'Runs SAST scan cfn_nag on the template'
         c.example 'run cfn_nag on stack myapp-vpc with us-east-1 settings', 'stack_master nag us-east-1 myapp-vpc'
         c.action do |args, options|
           options.default config: default_config_file
@@ -164,7 +168,7 @@ module StackMaster
 
       command :compile do |c|
         c.syntax = 'stack_master compile [region_or_alias] [stack_name]'
-        c.summary = "Print the compiled version of a given stack"
+        c.summary = 'Print the compiled version of a given stack'
         c.description = "Processes the stack and prints out a compiled version - same we'd send to AWS"
         c.example 'print compiled stack myapp-vpc with us-east-1 settings', 'stack_master compile us-east-1 myapp-vpc'
         c.action do |args, options|
@@ -181,7 +185,7 @@ module StackMaster
         c.example 'description', 'Check the status of all stack definitions'
         c.action do |args, options|
           options.default config: default_config_file
-          say "Invalid arguments. stack_master status" and return unless args.size == 0
+          say 'Invalid arguments. stack_master status' and return unless args.size == 0
 
           config = load_config(options.config)
           StackMaster::Commands::Status.perform(config, nil, options)
@@ -196,7 +200,7 @@ module StackMaster
         c.example 'description', 'Check for missing or extra files'
         c.action do |args, options|
           options.default config: default_config_file
-          say "Invalid arguments. stack_master tidy" and return unless args.size == 0
+          say 'Invalid arguments. stack_master tidy' and return unless args.size == 0
 
           config = load_config(options.config)
           StackMaster::Commands::Tidy.perform(config, nil, options)
@@ -211,7 +215,7 @@ module StackMaster
         c.action do |args, options|
           options.default config: default_config_file
           unless args.size == 2
-            say "Invalid arguments. stack_master delete [region] [stack_name]"
+            say 'Invalid arguments. stack_master delete [region] [stack_name]'
             return
           end
 
@@ -239,7 +243,7 @@ module StackMaster
         c.syntax = 'stack_master drift [region_or_alias] [stack_name]'
         c.summary = 'Detects and displays stack drift using the CloudFormation Drift API'
         c.description = 'Detects and displays stack drift'
-        c.option '--timeout SECONDS', Integer, "The number of seconds to wait for drift detection to complete"
+        c.option '--timeout SECONDS', Integer, 'The number of seconds to wait for drift detection to complete'
         c.example 'view stack drift for a stack named myapp-vpc in us-east-1', 'stack_master drift us-east-1 myapp-vpc'
         c.action do |args, options|
           options.default config: default_config_file, timeout: 120
@@ -253,7 +257,7 @@ module StackMaster
     private
 
     def default_config_file
-      "stack_master.yml"
+      'stack_master.yml'
     end
 
     def load_config(file)
@@ -277,10 +281,12 @@ module StackMaster
           show_other_region_candidates(config, stack_name)
           success = false
         end
-        stack_definitions = stack_definitions.select do |stack_definition|
-          running_in_allowed_account?(stack_definition.allowed_accounts) &&
-            StackStatus.new(config, stack_definition).changed?
-        end if options.changed
+        if options.changed
+          stack_definitions = stack_definitions.select do |stack_definition|
+            running_in_allowed_account?(stack_definition.allowed_accounts) &&
+              StackStatus.new(config, stack_definition).changed?
+          end
+        end
         stack_definitions.each do |stack_definition|
           StackMaster.cloud_formation_driver.set_region(stack_definition.region)
           StackMaster.stdout.puts(
@@ -295,14 +301,14 @@ module StackMaster
     end
 
     def show_other_region_candidates(config, stack_name)
-      candidates = config.filter(region = "", stack_name = stack_name)
+      candidates = config.filter(region = '', stack_name = stack_name)
       return if candidates.empty?
 
       StackMaster.stdout.puts "Stack name #{stack_name} exists in regions: #{candidates.map(&:region).join(', ')}"
     end
 
     def execute_if_allowed_account(allowed_accounts, &block)
-      raise ArgumentError, "Block required to execute this method" unless block_given?
+      raise ArgumentError, 'Block required to execute this method' unless block_given?
 
       if running_in_allowed_account?(allowed_accounts)
         block.call

data/lib/stack_master/command.rb

@@ -43,11 +43,7 @@ module StackMaster
       msg = "#{e.class} #{e.message}"
       msg << "\n Caused by: #{e.cause.class} #{e.cause.message}" if e.cause
       msg << "\n at #{e.cause.backtrace[0..3].join("\n    ")}\n ..." if e.cause && !options.trace
-      if options.trace
-        msg << "\n#{backtrace(e)}"
-      else
-        msg << "\n Use --trace to view backtrace"
-      end
+      msg << (options.trace ? "\n#{backtrace(e)}" : "\n Use --trace to view backtrace")
       msg
     end
 

data/lib/stack_master/commands/apply.rb

@@ -44,10 +44,10 @@ module StackMaster
       end
 
       def abort_if_review_in_progress
-        if stack_exists? && stack.stack_status == "REVIEW_IN_PROGRESS"
-          StackMaster.stderr.puts "Stack currently exists and is in #{stack.stack_status}"
-          failed! "You will need to delete the stack (#{stack.stack_name}) before continuing"
-        end
+        return unless stack_exists? && stack.stack_status == 'REVIEW_IN_PROGRESS'
+
+        StackMaster.stderr.puts "Stack currently exists and is in #{stack.stack_status}"
+        failed! "You will need to delete the stack (#{stack.stack_name}) before continuing"
       end
 
       def use_s3?
@@ -89,7 +89,7 @@ module StackMaster
           @change_set = ChangeSet.create(stack_options.merge(change_set_type: 'CREATE'))
           if @change_set.failed?
             ChangeSet.delete(@change_set.id)
-            halt!(@change_set.status_reason)
+            halt!(user_friendly_changeset_error(@change_set.status_reason))
           end
 
           @change_set.display(StackMaster.stdout)
@@ -111,11 +111,11 @@ module StackMaster
       end
 
       def ask_to_cancel_stack_update
-        if ask?("Cancel stack update?")
-          StackMaster.stdout.puts "Attempting to cancel stack update"
-          cf.cancel_update_stack(stack_name: stack_name)
-          tail_stack_events
-        end
+        return unless ask?('Cancel stack update?')
+
+        StackMaster.stdout.puts 'Attempting to cancel stack update'
+        cf.cancel_update_stack(stack_name: stack_name)
+        tail_stack_events
       end
 
       def update_stack
@@ -123,7 +123,7 @@ module StackMaster
         @change_set = ChangeSet.create(stack_options)
         if @change_set.failed?
           ChangeSet.delete(@change_set.id)
-          halt!(@change_set.status_reason)
+          halt!(user_friendly_changeset_error(@change_set.status_reason))
         end
 
         @change_set.display(StackMaster.stdout)
@@ -136,10 +136,10 @@ module StackMaster
       end
 
       def ask_update_confirmation!
-        unless ask?("Apply change set (y/n)? ")
-          ChangeSet.delete(@change_set.id)
-          halt! "Stack update aborted"
-        end
+        return if ask?('Apply change set (y/n)? ')
+
+        ChangeSet.delete(@change_set.id)
+        halt! 'Stack update aborted'
       end
 
       def upload_files
@@ -177,7 +177,7 @@ module StackMaster
           stack_name: stack_name,
           parameters: proposed_stack.aws_parameters,
           tags: proposed_stack.aws_tags,
-          capabilities: ['CAPABILITY_IAM', 'CAPABILITY_NAMED_IAM', 'CAPABILITY_AUTO_EXPAND'],
+          capabilities: %w[CAPABILITY_IAM CAPABILITY_NAMED_IAM CAPABILITY_AUTO_EXPAND],
           role_arn: proposed_stack.role_arn,
           notification_arns: proposed_stack.notification_arns,
           template_method => template_value
@@ -211,9 +211,9 @@ module StackMaster
       end
 
       def ensure_valid_template_body_size!
-        if proposed_stack.too_big?(use_s3?)
-          failed! TEMPLATE_TOO_LARGE_ERROR_MESSAGE
-        end
+        return unless proposed_stack.too_big?(use_s3?)
+
+        failed! TEMPLATE_TOO_LARGE_ERROR_MESSAGE
       end
 
       def set_stack_policy
@@ -230,6 +230,21 @@ module StackMaster
         StackMaster.stdout.puts 'done.'
       end
 
+      def user_friendly_changeset_error(status_reason)
+        # CloudFormation returns various messages when there are no changes to apply
+        if status_reason =~ /didn'?t contain changes|no changes|no updates are to be performed/i
+          <<~MESSAGE.chomp
+            #{status_reason}
+
+            While there may be differences in the template file (e.g., whitespace, comments, or
+            formatting), CloudFormation has determined that no actual resource changes are needed.
+            The stack is already in the desired state.
+          MESSAGE
+        else
+          status_reason
+        end
+      end
+
       extend Forwardable
       def_delegators :@stack_definition, :stack_name, :region
     end

data/lib/stack_master/commands/delete.rb

@@ -15,7 +15,7 @@ module StackMaster
         return unless check_exists
 
         unless ask?("Really delete stack #{@stack_name} (y/n)? ")
-          StackMaster.stdout.puts "Stack update aborted"
+          StackMaster.stdout.puts 'Stack update aborted'
           return
         end
 
@@ -33,7 +33,7 @@ module StackMaster
         cf.describe_stacks({ stack_name: @stack_name })
         true
       rescue Aws::CloudFormation::Errors::ValidationError
-        failed("Stack does not exist")
+        failed('Stack does not exist')
         false
       end
 
@@ -43,10 +43,10 @@ module StackMaster
 
       def tail_stack_events
         StackEvents::Streamer.stream(@stack_name, @region, io: StackMaster.stdout, from: @from_time)
-        StackMaster.stdout.puts "Stack deleted"
+        StackMaster.stdout.puts 'Stack deleted'
       rescue Aws::CloudFormation::Errors::ValidationError
         # Unfortunately the stack as a tendency of going away before we get the final delete event.
-        StackMaster.stdout.puts "Stack deleted"
+        StackMaster.stdout.puts 'Stack deleted'
       end
     end
   end

data/lib/stack_master/commands/drift.rb

@@ -6,9 +6,9 @@ module StackMaster
       include Command
       include Commander::UI
 
-      DETECTION_COMPLETE_STATES = [
-        'DETECTION_COMPLETE',
-        'DETECTION_FAILED'
+      DETECTION_COMPLETE_STATES = %w[
+        DETECTION_COMPLETE
+        DETECTION_FAILED
       ]
 
       def perform
@@ -41,9 +41,7 @@ module StackMaster
                        drift.physical_resource_id].join(' '), color)
         return unless drift.stack_resource_drift_status == 'MODIFIED'
 
-        unless drift.property_differences.empty?
-          puts colorize('  Property differences:', color)
-        end
+        puts colorize('  Property differences:', color) unless drift.property_differences.empty?
         drift.property_differences.each do |property_difference|
           puts colorize("  - #{property_difference.difference_type} #{property_difference.property_path}", color)
         end
@@ -98,9 +96,7 @@ module StackMaster
           break if DETECTION_COMPLETE_STATES.include?(resp.detection_status)
 
           elapsed_time = Time.now - start_time
-          if elapsed_time > @options.timeout
-            raise "Timeout waiting for stack drift detection"
-          end
+          raise 'Timeout waiting for stack drift detection' if elapsed_time > @options.timeout
 
           sleep SLEEP_SECONDS
         end

data/lib/stack_master/commands/events.rb

@@ -9,9 +9,9 @@ module StackMaster
         filter_events(events).each do |event|
           StackEvents::Presenter.print_event(StackMaster.stdout, event)
         end
-        if @options.tail
-          StackEvents::Streamer.stream(@stack_definition.stack_name, @stack_definition.region, io: StackMaster.stdout)
-        end
+        return unless @options.tail
+
+        StackEvents::Streamer.stream(@stack_definition.stack_name, @stack_definition.region, io: StackMaster.stdout)
       end
 
       private
@@ -22,9 +22,7 @@ module StackMaster
         else
           n = @options.number || 25
           from = events.count - n
-          if from < 0
-            from = 0
-          end
+          from = 0 if from < 0
           events[from..-1]
         end
       end

data/lib/stack_master/commands/init.rb

@@ -1,4 +1,4 @@
-require "erb"
+require 'erb'
 
 module StackMaster
   module Commands
@@ -12,22 +12,22 @@ module StackMaster
       end
 
       def perform
-        if check_files
-          create_stack_master_yml
-          create_stack_json_yml
-          create_parameters_yml
-        end
+        return unless check_files
+
+        create_stack_master_yml
+        create_stack_json_yml
+        create_parameters_yml
       end
 
       private
 
       def check_files
-        @stack_master_filename = "stack_master.yml"
+        @stack_master_filename = 'stack_master.yml'
         @stack_json_filename = "templates/#{@stack_name}.json"
-        @parameters_filename = File.join("parameters", "#{@stack_name}.yml")
-        @region_parameters_filename = File.join("parameters", @region, "#{@stack_name}.yml")
+        @parameters_filename = File.join('parameters', "#{@stack_name}.yml")
+        @region_parameters_filename = File.join('parameters', @region, "#{@stack_name}.yml")
 
-        if !@options.overwrite
+        unless @options.overwrite
           [@stack_master_filename, @stack_json_filename, @parameters_filename,
            @region_parameters_filename].each do |filename|
             next unless File.exist?(filename)
@@ -50,7 +50,7 @@ module StackMaster
       end
 
       def stack_json_template
-        File.join(StackMaster.base_dir, "stacktemplates", "stack.json.erb")
+        File.join(StackMaster.base_dir, 'stacktemplates', 'stack.json.erb')
       end
 
       def create_stack_master_yml
@@ -63,7 +63,7 @@ module StackMaster
       end
 
       def stack_master_template
-        File.join(StackMaster.base_dir, "stacktemplates", "stack_master.yml.erb")
+        File.join(StackMaster.base_dir, 'stacktemplates', 'stack_master.yml.erb')
       end
 
       def create_parameters_yml
@@ -83,11 +83,11 @@ module StackMaster
       end
 
       def parameter_stack_name_template
-        File.join(StackMaster.base_dir, "stacktemplates", "parameter_stack_name.yml")
+        File.join(StackMaster.base_dir, 'stacktemplates', 'parameter_stack_name.yml')
       end
 
       def parameter_region_template
-        File.join(StackMaster.base_dir, "stacktemplates", "parameter_region.yml")
+        File.join(StackMaster.base_dir, 'stacktemplates', 'parameter_region.yml')
       end
 
       def render(renderer)

data/lib/stack_master/commands/lint.rb

@@ -19,7 +19,7 @@ module StackMaster
           f.write(proposed_stack.template_body)
           f.flush
           system('cfn-lint', f.path)
-          puts "cfn-lint run complete"
+          puts 'cfn-lint run complete'
         end
       end
 

data/lib/stack_master/commands/list_stacks.rb

@@ -8,7 +8,7 @@ module StackMaster
       include StackMaster::Commands::TerminalHelper
 
       def perform
-        tp.set :max_width, self.window_size
+        tp.set :max_width, window_size
         tp @config.stacks, :region, :stack_name
       end
     end

data/lib/stack_master/commands/outputs.rb

@@ -9,7 +9,7 @@ module StackMaster
 
       def perform
         if stack
-          tp.set :max_width, self.window_size
+          tp.set :max_width, window_size
           tp stack.outputs, :output_key, :output_value, :description
         else
           failed("Stack doesn't exist")

data/lib/stack_master/commands/status.rb

@@ -21,11 +21,11 @@ module StackMaster
           {
             region: stack_definition.region,
             stack_name: stack_definition.stack_name,
-            stack_status: running_in_allowed_account?(allowed_accounts) ? stack_status.status : "Disallowed account",
-            different: running_in_allowed_account?(allowed_accounts) ? stack_status.changed_message : "N/A",
+            stack_status: running_in_allowed_account?(allowed_accounts) ? stack_status.status : 'Disallowed account',
+            different: running_in_allowed_account?(allowed_accounts) ? stack_status.changed_message : 'N/A'
           }
         end
-        tp.set :max_width, self.window_size
+        tp.set :max_width, window_size
         tp.set :io, StackMaster.stdout
         tp status
         StackMaster.stdout.puts " * No echo parameters can't be diffed"
@@ -34,7 +34,7 @@ module StackMaster
       private
 
       def progress
-        @progress ||= ProgressBar.create(title: "Fetching stack information",
+        @progress ||= ProgressBar.create(title: 'Fetching stack information',
                                          total: @config.stacks.size,
                                          output: StackMaster.stdout)
       end