stack_master 2.17.0 → 2.18.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 637daba2dddc707df22584fad412d87df4ee24aab069885231561d847c762496
-  data.tar.gz: 7abcea21ecfbc080886b4181dc66392066a9360255ed2186bf2e7d652f6101e2
+  metadata.gz: ffa37da4e3a6d6d80804a60be1b868180a67470009472b02a0c838354028efb6
+  data.tar.gz: e04ddf265d427425f0a1a60973eec74e2e6cc42e3cc47f00b8cbf584f1cdac93
 SHA512:
-  metadata.gz: dba94d8ecf220d837673ef95f68c8d534b979bd31d67b8056e0836355a88f713499ca55420906ffb16855f7c3acb17e320c41ec90cb9c84263c59539f00c4cb2
-  data.tar.gz: 680f31e8fef7a94d387582acdb6c9381844aae4e2c56e6c4088f36d9c1ac2969feb27b9703c90c21416cef96572efc32455d4621951567611a18cb3481f22f56
+  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:
 
@@ -794,11 +794,6 @@ CLI directly rather than via a stack update.
 applied in CloudFormation. This can happen if the template or computed parameters have changed in code and the change
 hasn't been applied to this stack.
 
-## Maintainers
-
-- [Steve Hodgkiss](https://github.com/stevehodgkiss)
-- [Glen Stampoultzis](https://github.com/gstamp)
-
 ## License
 
 StackMaster uses the MIT license. See [LICENSE.txt](https://github.com/envato/stack_master/blob/master/LICENSE.txt) for details.

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,37 +8,39 @@ 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 :cf, :create_change_set,
-                          :describe_change_set,
-                          :execute_change_set,
-                          :delete_change_set,
-                          :delete_stack,
-                          :cancel_update_stack,
-                          :describe_stack_resources,
-                          :get_template,
-                          :get_stack_policy,
-                          :set_stack_policy,
-                          :describe_stack_events,
-                          :update_stack,
-                          :create_stack,
-                          :validate_template,
-                          :describe_stacks,
-                          :detect_stack_drift,
-                          :describe_stack_drift_detection_status,
-                          :describe_stack_resource_drifts
+      def_delegators(
+        :cf,
+        :create_change_set,
+        :describe_change_set,
+        :execute_change_set,
+        :delete_change_set,
+        :delete_stack,
+        :cancel_update_stack,
+        :describe_stack_resources,
+        :get_template,
+        :get_stack_policy,
+        :set_stack_policy,
+        :describe_stack_events,
+        :update_stack,
+        :create_stack,
+        :validate_template,
+        :describe_stacks,
+        :detect_stack_drift,
+        :describe_stack_drift_detection_status,
+        :describe_stack_resource_drifts
+      )
 
       private
 
       def cf
         @cf ||= Aws::CloudFormation::Client.new({ region: region, retry_limit: 10 })
       end
-
     end
   end
 end

data/lib/stack_master/aws_driver/s3.rb

@@ -11,20 +11,24 @@ module StackMaster
       end
 
       def upload_files(bucket: nil, prefix: nil, region: nil, files: {})
-        raise StackMaster::AwsDriver::S3ConfigurationError, 'A bucket must be specified in order to use S3' unless bucket
+        unless bucket
+          raise StackMaster::AwsDriver::S3ConfigurationError, 'A bucket must be specified in order to use S3'
+        end
 
         return if files.empty?
 
         s3 = new_s3_client(region: region)
 
-        current_objects = s3.list_objects({
-          prefix: prefix,
-          bucket: bucket
-        }).map(&:contents).flatten.inject({}){|h,obj|
+        current_objects = s3.list_objects(
+          {
+            prefix: prefix,
+            bucket: bucket
+          }
+        ).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)
@@ -32,26 +36,29 @@ 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
+
           s3_uri = "s3://#{bucket}/#{object_key}"
           StackMaster.stdout.print "- #{File.basename(path)} => #{s3_uri} "
 
-          s3.put_object({
-            bucket: bucket,
-            key: object_key,
-            body: body,
-            metadata: { md5: compiled_template_md5 }
-          })
-          StackMaster.stdout.puts "done."
+          s3.put_object(
+            {
+              bucket: bucket,
+              key: object_key,
+              body: body,
+              metadata: { md5: compiled_template_md5 }
+            }
+          )
+          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)
@@ -42,15 +42,15 @@ module StackMaster
     end
 
     def display(io)
-      io.puts <<-EOL
+      io.puts <<~EOL
 
-========================================
-Proposed change set:
-EOL
+        ========================================
+        Proposed change set:
+      EOL
       @response.changes.each do |change|
         display_resource_change(io, change.resource_change)
       end
-io.puts "========================================"
+      io.puts '========================================'
     end
 
     def failed?
@@ -84,13 +84,9 @@ io.puts "========================================"
     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

@@ -5,8 +5,12 @@ module StackMaster
   class CLI
     include Commander::Methods
 
-    def initialize(argv, stdin=STDIN, stdout=STDOUT, stderr=STDERR, kernel=Kernel)
-      @argv, @stdin, @stdout, @stderr, @kernel = argv, stdin, stdout, stderr, kernel
+    def initialize(argv, stdin = STDIN, stdout = STDOUT, stderr = STDERR, kernel = 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
@@ -41,10 +45,15 @@ module StackMaster
       command :apply do |c|
         c.syntax = 'stack_master apply [region_or_alias] [stack_name]'
         c.summary = 'Creates or updates a stack'
-        c.description = "Creates or updates a stack. Shows a diff of the proposed stack's template and parameters. Tails stack events until CloudFormation has completed."
+        c.description = "Creates or updates a stack. Shows a diff of the proposed stack's template and parameters. " \
+                        'Tails stack events until CloudFormation has completed.'
         c.example 'update a stack named myapp-vpc in us-east-1', 'stack_master apply us-east-1 myapp-vpc'
-        c.option '--on-failure ACTION', String, "Action to take on CREATE_FAILURE. Valid Values: [ DO_NOTHING | ROLLBACK | DELETE ]. Default: ROLLBACK\nNote: 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 '--on-failure ACTION', String,
+                 'Action to take on CREATE_FAILURE. ' \
+                 '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.action do |args, options|
           options.default config: default_config_file
           execute_stacks_command(StackMaster::Commands::Apply, args, options)
@@ -54,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)
@@ -68,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
@@ -89,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'
@@ -103,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)
@@ -117,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
@@ -137,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
@@ -149,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
@@ -159,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|
@@ -171,11 +180,13 @@ module StackMaster
       command :status do |c|
         c.syntax = 'stack_master status'
         c.summary = 'Check the current status stacks.'
-        c.description = 'Checks the status of all stacks defined in the stack_master.yml file. Warning this operation can be somewhat slow.'
+        c.description = 'Checks the status of all stacks defined in the stack_master.yml file. ' \
+                        'Warning this operation can be somewhat slow.'
         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)
         end
@@ -184,11 +195,13 @@ module StackMaster
       command :tidy do |c|
         c.syntax = 'stack_master tidy'
         c.summary = 'Try to identify extra & missing files.'
-        c.description = 'Cross references stack_master.yml with the template and parameter directories to identify extra or missing files.'
+        c.description = 'Cross references stack_master.yml with the template ' \
+                        'and parameter directories to identify extra or missing files.'
         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)
         end
@@ -202,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
 
@@ -230,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
@@ -244,7 +257,7 @@ module StackMaster
     private
 
     def default_config_file
-      "stack_master.yml"
+      'stack_master.yml'
     end
 
     def load_config(file)
@@ -268,12 +281,17 @@ 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 "Executing #{command.command_name} on #{stack_definition.stack_name} in #{stack_definition.region}"
+          StackMaster.stdout.puts(
+            "Executing #{command.command_name} on #{stack_definition.stack_name} in #{stack_definition.region}"
+          )
           success = execute_if_allowed_account(stack_definition.allowed_accounts) do
             command.perform(config, stack_definition, options).success?
           end
@@ -283,20 +301,23 @@ 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
       else
         account_text = "'#{identity.account}'"
         account_text << " (#{identity.account_aliases.join(', ')})" if identity.account_aliases.any?
-        StackMaster.stdout.puts "Account #{account_text} is not an allowed account. Allowed accounts are #{allowed_accounts}."
+        StackMaster.stdout.puts(
+          "Account #{account_text} is not an allowed account. Allowed accounts are #{allowed_accounts}."
+        )
         false
       end
     end

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,14 +136,15 @@ 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
         return unless use_s3?
+
         s3.upload_files(**s3_options)
       end
 
@@ -153,7 +154,8 @@ module StackMaster
 
       def template_value
         if use_s3?
-          s3.url(bucket: @s3_config['bucket'], prefix: @s3_config['prefix'], region: @s3_config['region'], template: @stack_definition.s3_template_file_name)
+          s3.url(bucket: @s3_config['bucket'], prefix: @s3_config['prefix'], region: @s3_config['region'],
+                 template: @stack_definition.s3_template_file_name)
         else
           proposed_stack.template
         end
@@ -161,6 +163,7 @@ module StackMaster
 
       def files_to_upload
         return {} unless use_s3?
+
         @stack_definition.s3_files.tap do |files|
           files[@stack_definition.s3_template_file_name] = {
             path: @stack_definition.template_file_path,
@@ -174,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
@@ -208,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
@@ -218,6 +221,7 @@ module StackMaster
         proposed_policy = proposed_stack.stack_policy_body
         # No need to reset a stack policy if it's nil or not changed
         return if proposed_policy.nil? || proposed_policy == current_policy
+
         StackMaster.stdout.print 'Setting a stack policy...'
         cf.set_stack_policy(
           stack_name: stack_name,
@@ -226,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

@@ -12,11 +12,10 @@ module StackMaster
       end
 
       def perform
-
         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
 
@@ -27,14 +26,14 @@ module StackMaster
       private
 
       def delete_stack
-        cf.delete_stack({stack_name: @stack_name})
+        cf.delete_stack({ stack_name: @stack_name })
       end
 
       def check_exists
-        cf.describe_stacks({stack_name: @stack_name})
+        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
 
@@ -44,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