fastlane 2.74.1 → 2.75.0.beta.20180109010003

data/frameit/lib/frameit/editor.rb

@@ -59,7 +59,7 @@ module Frameit
       output_path = screenshot.path.gsub('.png', '_framed.png').gsub('.PNG', '_framed.png')
       image.format("png")
       image.write(output_path)
-      UI.success "Added frame: '#{File.expand_path(output_path)}'"
+      UI.success("Added frame: '#{File.expand_path(output_path)}'")
     end
 
     # puts the screenshot into the frame
@@ -72,8 +72,8 @@ module Frameit
       @image.rotate(-rotation)
 
       @image = frame.composite(image, "png") do |c|
-        c.compose "DstOver"
-        c.geometry offset['offset']
+        c.compose("DstOver")
+        c.geometry(offset['offset'])
       end
 
       # Revert the rotation from above
@@ -89,7 +89,7 @@ module Frameit
       if @offset_information and (@offset_information['offset'] or @offset_information['offset'])
         return @offset_information
       end
-      UI.user_error! "Could not find offset_information for '#{screenshot}'"
+      UI.user_error!("Could not find offset_information for '#{screenshot}'")
     end
 
     #########################################################################################
@@ -139,11 +139,11 @@ module Frameit
           image_width = [frame_width, @image.width].min
           image_height = [frame_height, image_width / image_aspect_ratio].min
           image_width = image_height * image_aspect_ratio
-          @image.resize "#{image_width}x#{image_height}" if image_width < @image.width || image_height < @image.height
+          @image.resize("#{image_width}x#{image_height}") if image_width < @image.width || image_height < @image.height
         else
           # the screenshot size is only limited by width.
           # If higher than the frame, the screenshot is cut off at the bottom
-          @image.resize "#{frame_width}x" if frame_width < @image.width
+          @image.resize("#{frame_width}x") if frame_width < @image.width
         end
       end
 
@@ -184,8 +184,8 @@ module Frameit
       background = MiniMagick::Image.open(fetch_config['background'])
 
       if background.height != screenshot.size[1]
-        background.resize "#{screenshot.size[0]}x#{screenshot.size[1]}^" # `^` says it should fill area
-        background.merge! ["-gravity", "center", "-crop", "#{screenshot.size[0]}x#{screenshot.size[1]}+0+0"] # crop from center
+        background.resize("#{screenshot.size[0]}x#{screenshot.size[1]}^") # `^` says it should fill area
+        background.merge!(["-gravity", "center", "-crop", "#{screenshot.size[0]}x#{screenshot.size[1]}+0+0"]) # crop from center
       end
       background
     end
@@ -195,13 +195,13 @@ module Frameit
       title_below_image = fetch_config['title_below_image']
 
       @image = background.composite(image, "png") do |c|
-        c.compose "Over"
+        c.compose("Over")
         if title_below_image
           show_complete_frame = fetch_config['show_complete_frame']
-          c.geometry "+#{left_space}+#{background.height - image.height - space_to_device}" unless show_complete_frame
-          c.geometry "+#{left_space}+#{vertical_frame_padding}" if show_complete_frame
+          c.geometry("+#{left_space}+#{background.height - image.height - space_to_device}") unless show_complete_frame
+          c.geometry("+#{left_space}+#{vertical_frame_padding}") if show_complete_frame
         else
-          c.geometry "+#{left_space}+#{space_to_device}"
+          c.geometry("+#{left_space}+#{space_to_device}")
         end
       end
 
@@ -214,7 +214,7 @@ module Frameit
 
       multiplicator = (screenshot_width.to_f / offset['width'].to_f) # by how much do we have to change this?
       new_frame_width = multiplicator * frame.width # the new width for the frame
-      frame.resize "#{new_frame_width.round}x" # resize it to the calculated width
+      frame.resize("#{new_frame_width.round}x") # resize it to the calculated width
       modify_offset(multiplicator) # modify the offset to properly insert the screenshot into the frame later
     end
 
@@ -224,7 +224,7 @@ module Frameit
       if ratio > 1.0
         # too large - resizing now
         smaller = (1.0 / ratio)
-        text.resize "#{(smaller * text.width).round}x"
+        text.resize("#{(smaller * text.width).round}x")
       end
     end
 
@@ -248,16 +248,16 @@ module Frameit
       title_below_image = fetch_config['title_below_image']
       # keyword
       background = background.composite(keyword, "png") do |c|
-        c.compose "Over"
-        c.geometry "+#{keyword_left_space}+#{keyword_top_space}" unless title_below_image
-        c.geometry "+#{keyword_left_space}+#{background.height - space_to_device + keyword_top_space}" if title_below_image
+        c.compose("Over")
+        c.geometry("+#{keyword_left_space}+#{keyword_top_space}") unless title_below_image
+        c.geometry("+#{keyword_left_space}+#{background.height - space_to_device + keyword_top_space}") if title_below_image
       end
       # Then, put the title on top of the screenshot next to the keyword
       # Then, put the title above/below of the screenshot next to the keyword
       background = background.composite(title, "png") do |c|
-        c.compose "Over"
-        c.geometry "+#{title_left_space}+#{title_top_space}" unless title_below_image
-        c.geometry "+#{title_left_space}+#{background.height - space_to_device + title_top_space}" if title_below_image
+        c.compose("Over")
+        c.geometry("+#{title_left_space}+#{title_top_space}") unless title_below_image
+        c.geometry("+#{title_left_space}+#{background.height - space_to_device + title_top_space}") if title_below_image
       end
       background
     end
@@ -286,8 +286,8 @@ module Frameit
 
         UI.verbose("Text for image #{self.screenshot.path} is quite long, reducing font size by #{(ratio - 1.0).round(2)}")
 
-        title.resize "#{(smaller * title.width).round}x"
-        keyword.resize "#{(smaller * keyword.width).round}x" if keyword
+        title.resize("#{(smaller * title.width).round}x")
+        keyword.resize("#{(smaller * keyword.width).round}x") if keyword
         sum_width *= smaller
       end
 
@@ -301,9 +301,9 @@ module Frameit
       # First, put the keyword on top of the screenshot, if we have one
       if keyword
         background = background.composite(keyword, "png") do |c|
-          c.compose "Over"
-          c.geometry "+#{left_space}+#{top_space}" unless title_below_image
-          c.geometry "+#{left_space}+#{background.height - space_to_device + top_space}" if title_below_image
+          c.compose("Over")
+          c.geometry("+#{left_space}+#{top_space}") unless title_below_image
+          c.geometry("+#{left_space}+#{background.height - space_to_device + top_space}") if title_below_image
         end
 
         left_space += keyword.width + (keyword_padding * smaller)
@@ -311,9 +311,9 @@ module Frameit
 
       # Then, put the title on top of the screenshot next to the keyword
       background = background.composite(title, "png") do |c|
-        c.compose "Over"
-        c.geometry "+#{left_space}+#{top_space}" unless title_below_image
-        c.geometry "+#{left_space}+#{background.height - space_to_device + top_space}" if title_below_image
+        c.compose("Over")
+        c.geometry("+#{left_space}+#{top_space}") unless title_below_image
+        c.geometry("+#{left_space}+#{background.height - space_to_device + top_space}") if title_below_image
       end
       background
     end
@@ -342,7 +342,7 @@ module Frameit
         image_height = max_height # gets trimmed afterwards anyway, and on the iPad the `y` would get cut
         title_image.combine_options do |i|
           # Oversize as the text might be larger than the actual image. We're trimming afterwards anyway
-          i.resize "#{max_width * 5.0}x#{image_height}!" # `!` says it should ignore the ratio
+          i.resize("#{max_width * 5.0}x#{image_height}!") # `!` says it should ignore the ratio
         end
 
         current_font = font(key)
@@ -350,19 +350,19 @@ module Frameit
         UI.verbose("Using #{current_font} as font the #{key} of #{screenshot.path}") if current_font
         UI.verbose("Adding text '#{text}'")
 
-        text.gsub! '\n', "\n"
+        text.gsub!('\n', "\n")
         text.gsub!(/(?<!\\)(')/) { |s| "\\#{s}" } # escape unescaped apostrophes with a backslash
 
         interline_spacing = fetch_config['interline_spacing']
 
         # Add the actual title
         title_image.combine_options do |i|
-          i.font current_font if current_font
-          i.gravity "Center"
-          i.pointsize actual_font_size
-          i.draw "text 0,0 '#{text}'"
-          i.interline_spacing interline_spacing if interline_spacing
-          i.fill fetch_config[key.to_s]['color']
+          i.font(current_font) if current_font
+          i.gravity("Center")
+          i.pointsize(actual_font_size)
+          i.draw("text 0,0 '#{text}'")
+          i.interline_spacing(interline_spacing) if interline_spacing
+          i.fill(fetch_config[key.to_s]['color'])
         end
 
         results[key] = title_image
@@ -435,13 +435,13 @@ module Frameit
 
     # Fetches the title + keyword for this particular screenshot
     def fetch_text(type)
-      UI.user_error! "Valid parameters :keyword, :title" unless [:keyword, :title].include? type
+      UI.user_error!("Valid parameters :keyword, :title") unless [:keyword, :title].include?(type)
 
       # Try to get it from a keyword.strings or title.strings file
       strings_path = File.join(File.expand_path("..", screenshot.path), "#{type}.strings")
-      if File.exist? strings_path
+      if File.exist?(strings_path)
         parsed = StringsParser.parse(strings_path)
-        result = parsed.find { |k, v| screenshot.path.upcase.include? k.upcase }
+        result = parsed.find { |k, v| screenshot.path.upcase.include?(k.upcase) }
         return result.last if result
       end
 
@@ -451,7 +451,7 @@ module Frameit
 
       if type == :title and !result
         # title is mandatory
-        UI.user_error! "Could not get title for screenshot #{screenshot.path}. Please provide one in your Framefile.json"
+        UI.user_error!("Could not get title for screenshot #{screenshot.path}. Please provide one in your Framefile.json")
       end
 
       return result
@@ -467,7 +467,7 @@ module Frameit
         fonts.each do |font|
           if font['supported']
             font['supported'].each do |language|
-              if screenshot.path.include? language
+              if screenshot.path.include?(language)
                 return font["font"]
               end
             end

data/frameit/lib/frameit/frame_downloader.rb

@@ -52,18 +52,18 @@ module Frameit
     end
 
     def print_disclaimer
-      UI.header "Device frames disclaimer"
-      UI.important "All used device frames are available via Facebook Design: http://facebook.design/devices"
-      UI.message "----------------------------------------"
-      UI.message "While Facebook has redrawn and shares these assets for the benefit"
-      UI.message "of the design community, Facebook does not own any of the underlying"
-      UI.message "product or user interface designs."
-      UI.message "By accessing these assets, you agree to obtain all necessary permissions"
-      UI.message "from the underlying rights holders and/or adhere to any applicable brand"
-      UI.message "use guidelines before using them."
-      UI.message "Facebook disclaims all express or implied warranties with respect to these assets, including"
-      UI.message "non-infringement of intellectual property rights."
-      UI.message "----------------------------------------"
+      UI.header("Device frames disclaimer")
+      UI.important("All used device frames are available via Facebook Design: http://facebook.design/devices")
+      UI.message("----------------------------------------")
+      UI.message("While Facebook has redrawn and shares these assets for the benefit")
+      UI.message("of the design community, Facebook does not own any of the underlying")
+      UI.message("product or user interface designs.")
+      UI.message("By accessing these assets, you agree to obtain all necessary permissions")
+      UI.message("from the underlying rights holders and/or adhere to any applicable brand")
+      UI.message("use guidelines before using them.")
+      UI.message("Facebook disclaims all express or implied warranties with respect to these assets, including")
+      UI.message("non-infringement of intellectual property rights.")
+      UI.message("----------------------------------------")
     end
 
     private

data/frameit/lib/frameit/mac_editor.rb

@@ -13,8 +13,8 @@ module Frameit
       self.top_space_above_device = offset['titleHeight'] # needed for centering the title
 
       @image = background.composite(image, "png") do |c|
-        c.compose "Over"
-        c.geometry offset['offset']
+        c.compose("Over")
+        c.geometry(offset['offset'])
       end
 
       return image

data/frameit/lib/frameit/options.rb

@@ -30,7 +30,7 @@ module Frameit
                                      optional: true,
                                      verify_block: proc do |value|
                                        available = ['iPhone_6_Plus', 'iPhone_5s', 'iPhone_4', 'iPad_mini', 'Mac']
-                                       unless available.include? value
+                                       unless available.include?(value)
                                          UI.user_error!("Invalid device type '#{value}'. Available values: #{available}")
                                        end
                                      end),

data/frameit/lib/frameit/runner.rb

@@ -26,11 +26,11 @@ module Frameit
 
       if screenshots.count > 0
         screenshots.each do |full_path|
-          next if full_path.include? "_framed.png"
-          next if full_path.include? ".itmsp/" # a package file, we don't want to modify that
-          next if full_path.include? "device_frames/" # these are the device frames the user is using
+          next if full_path.include?("_framed.png")
+          next if full_path.include?(".itmsp/") # a package file, we don't want to modify that
+          next if full_path.include?("device_frames/") # these are the device frames the user is using
           device = full_path.rpartition('/').last.partition('-').first # extract device name
-          if device.downcase.include? "watch"
+          if device.downcase.include?("watch")
             UI.error("Apple Watch screenshots are not framed: '#{full_path}'")
             next # we don't care about watches right now
           end
@@ -41,12 +41,12 @@ module Frameit
             screenshot = Screenshot.new(full_path, color)
             screenshot.frame!
           rescue => ex
-            UI.error ex.to_s
-            UI.error "Backtrace:\n\t#{ex.backtrace.join("\n\t")}" if FastlaneCore::Globals.verbose?
+            UI.error(ex.to_s)
+            UI.error("Backtrace:\n\t#{ex.backtrace.join("\n\t")}") if FastlaneCore::Globals.verbose?
           end
         end
       else
-        UI.error "Could not find screenshots in current directory: '#{File.expand_path(path)}'"
+        UI.error("Could not find screenshots in current directory: '#{File.expand_path(path)}'")
       end
     end
   end

data/frameit/lib/frameit/screenshot.rb

@@ -16,7 +16,7 @@ module Frameit
     # path: Path to screenshot
     # color: Color to use for the frame
     def initialize(path, color)
-      UI.user_error!("Couldn't find file at path '#{path}'") unless File.exist? path
+      UI.user_error!("Couldn't find file at path '#{path}'") unless File.exist?(path)
       @color = color
       @path = path
       @size = FastImage.size(path)
@@ -26,7 +26,7 @@ module Frameit
 
     # Device name for a given screen size. Used to use the correct template
     def device_name
-      # rubocop:disable Lint/MissingRequireStatement
+      # rubocop:disable Require/MissingRequireStatement
       sizes = Deliver::AppScreenshot::ScreenSize
       case @screen_size
       when sizes::IOS_58
@@ -46,9 +46,9 @@ module Frameit
       when sizes::MAC
         return 'MacBook'
       else
-        UI.error "Unknown device type for size #{@screen_size} for path '#{path}'"
+        UI.error("Unknown device type for size #{@screen_size} for path '#{path}'")
       end
-      # rubocop:enable Lint/MissingRequireStatement
+      # rubocop:enable Require/MissingRequireStatement
     end
 
     def color
@@ -89,7 +89,7 @@ module Frameit
         valid = [:landscape_left, :landscape_right, :portrait, nil]
         UI.user_error("orientation_block must return #{valid[0..-2].join(', ')} or nil") unless valid.include?(orientation)
 
-        puts "Forced orientation: #{orientation}"
+        puts("Forced orientation: #{orientation}")
       end
 
       return orientation unless orientation.nil?

data/frameit/lib/frameit/strings_parser.rb

@@ -4,8 +4,8 @@ module Frameit
   # This class will parse the .string files
   class StringsParser
     def self.parse(path)
-      UI.user_error! "Couldn't find strings file at path '#{path}'" unless File.exist? path
-      UI.user_error! "Must be .strings file, only got '#{path}'" unless path.end_with? ".strings"
+      UI.user_error!("Couldn't find strings file at path '#{path}'") unless File.exist?(path)
+      UI.user_error!("Must be .strings file, only got '#{path}'") unless path.end_with?(".strings")
 
       result = {}
 
@@ -15,20 +15,20 @@ module Frameit
       content.split("\n").each_with_index do |line, index|
         begin
           # We don't care about comments and empty lines
-          if line.start_with? '"'
+          if line.start_with?('"')
             key = line.match(/"(.*)" \= /)[1]
             value = line.match(/ \= "(.*)"/)[1]
 
             result[key] = value
           end
         rescue => ex
-          UI.error "Error parsing #{path} line #{index + 1}: '#{line}'"
-          UI.verbose "#{ex.message}\n#{ex.backtrace.join('\n')}"
+          UI.error("Error parsing #{path} line #{index + 1}: '#{line}'")
+          UI.verbose("#{ex.message}\n#{ex.backtrace.join('\n')}")
         end
       end
 
       if result.empty?
-        UI.error "Empty parsing result for #{path}. Please make sure the file is valid and UTF16 Big-endian encoded"
+        UI.error("Empty parsing result for #{path}. Please make sure the file is valid and UTF16 Big-endian encoded")
       end
 
       result

data/frameit/lib/frameit/template_finder.rb

@@ -15,12 +15,12 @@ module Frameit
 
       templates = Dir["#{FrameDownloader.templates_path}/#{filename}.{png,jpg}"] # ~/.frameit folder
 
-      UI.verbose "Looking for #{filename} and found #{templates.count} template(s)"
+      UI.verbose("Looking for #{filename} and found #{templates.count} template(s)")
 
       if templates.count == 0
         if screenshot.screen_size == Deliver::AppScreenshot::ScreenSize::IOS_35
-          UI.important "Unfortunately 3.5\" device frames were discontinued. Skipping screen '#{screenshot.path}'"
-          UI.error "Looked for: '#{filename}.png'"
+          UI.important("Unfortunately 3.5\" device frames were discontinued. Skipping screen '#{screenshot.path}'")
+          UI.error("Looked for: '#{filename}.png'")
         elsif screenshot.color == Frameit::Color::ROSE_GOLD || screenshot.color == Frameit::Color::GOLD
           # Unfortunately not every device type is available in rose gold or gold
           # This is why we can't have nice things #yatusabes

data/gym/lib/gym/commands_generator.rb

@@ -51,12 +51,12 @@ module Gym
         c.action do |args, options|
           containing = FastlaneCore::Helper.fastlane_enabled_folder_path
           path = File.join(containing, Gym.gymfile_name)
-          UI.user_error! "Gymfile already exists" if File.exist?(path)
+          UI.user_error!("Gymfile already exists") if File.exist?(path)
 
           is_swift_fastfile = args.include?("swift")
           if is_swift_fastfile
             path = File.join(containing, Gym.gymfile_name + ".swift")
-            UI.user_error! "Gymfile.swift already exists" if File.exist?(path)
+            UI.user_error!("Gymfile.swift already exists") if File.exist?(path)
           end
 
           if is_swift_fastfile
@@ -66,11 +66,11 @@ module Gym
           end
 
           File.write(path, template)
-          UI.success "Successfully created '#{path}'. Open the file using a code editor."
+          UI.success("Successfully created '#{path}'. Open the file using a code editor.")
         end
       end
 
-      default_command :build
+      default_command(:build)
 
       run!
     end

data/gym/lib/gym/detect_values.rb

@@ -126,7 +126,7 @@ module Gym
       if config[:configuration]
         # Verify the configuration is available
         unless configurations.include?(config[:configuration])
-          UI.error "Couldn't find specified configuration '#{config[:configuration]}'."
+          UI.error("Couldn't find specified configuration '#{config[:configuration]}'.")
           config[:configuration] = nil
         end
       end

data/gym/lib/gym/error_handler.rb

@@ -13,46 +13,46 @@ module Gym
         # The order of the handling below is important
         case output
         when /Your build settings specify a provisioning profile with the UUID/
-          print "Invalid code signing settings"
-          print "Your project defines a provisioning profile which doesn't exist on your local machine"
-          print "You can use sigh (https://docs.fastlane.tools/actions/sigh/) to download and install the provisioning profile"
-          print "Follow this guide: https://docs.fastlane.tools/codesigning/GettingStarted/"
+          print("Invalid code signing settings")
+          print("Your project defines a provisioning profile which doesn't exist on your local machine")
+          print("You can use sigh (https://docs.fastlane.tools/actions/sigh/) to download and install the provisioning profile")
+          print("Follow this guide: https://docs.fastlane.tools/codesigning/GettingStarted/")
         when /Provisioning profile does not match bundle identifier/
-          print "Invalid code signing settings"
-          print "Your project defines a provisioning profile that doesn't match the bundle identifier of your app"
-          print "Make sure you use the correct provisioning profile for this app"
-          print "Take a look at the ouptput above for more information"
-          print "You can follow this guide: https://docs.fastlane.tools/codesigning/GettingStarted/"
+          print("Invalid code signing settings")
+          print("Your project defines a provisioning profile that doesn't match the bundle identifier of your app")
+          print("Make sure you use the correct provisioning profile for this app")
+          print("Take a look at the ouptput above for more information")
+          print("You can follow this guide: https://docs.fastlane.tools/codesigning/GettingStarted/")
         when /provisioning profiles matching the bundle identifier .(.*)./ # the . around the (.*) are for the strange "
-          print "You don't have the provisioning profile for '#{$1}' installed on the local machine"
-          print "Make sure you have the profile on this computer and it's properly installed"
-          print "You can use sigh (https://docs.fastlane.tools/actions/sigh/) to download and install the provisioning profile"
-          print "Follow this guide: https://docs.fastlane.tools/codesigning/GettingStarted/"
+          print("You don't have the provisioning profile for '#{$1}' installed on the local machine")
+          print("Make sure you have the profile on this computer and it's properly installed")
+          print("You can use sigh (https://docs.fastlane.tools/actions/sigh/) to download and install the provisioning profile")
+          print("Follow this guide: https://docs.fastlane.tools/codesigning/GettingStarted/")
         when /matching the bundle identifier .(.*). were found/ # the . around the (.*) are for the strange "
-          print "You don't have a provisioning profile for the bundle identifier '#{$1}' installed on the local machine"
-          print "Make sure you have the profile on this computer and it's properly installed"
-          print "You can use sigh (https://docs.fastlane.tools/actions/sigh/) to download and install the provisioning profile"
-          print "Follow this guide: https://docs.fastlane.tools/codesigning/GettingStarted/"
+          print("You don't have a provisioning profile for the bundle identifier '#{$1}' installed on the local machine")
+          print("Make sure you have the profile on this computer and it's properly installed")
+          print("You can use sigh (https://docs.fastlane.tools/actions/sigh/) to download and install the provisioning profile")
+          print("Follow this guide: https://docs.fastlane.tools/codesigning/GettingStarted/")
 
         # Insert more code signing specific errors here
         when /code signing is required/
-          print "Your project settings define invalid code signing settings"
-          print "To generate an ipa file you need to enable code signing for your project"
-          print "Additionally make sure you have a code signing identity set"
-          print "Follow this guide: https://docs.fastlane.tools/codesigning/GettingStarted/"
+          print("Your project settings define invalid code signing settings")
+          print("To generate an ipa file you need to enable code signing for your project")
+          print("Additionally make sure you have a code signing identity set")
+          print("Follow this guide: https://docs.fastlane.tools/codesigning/GettingStarted/")
         when /US\-ASCII/
-          print "Your shell environment is not correctly configured"
-          print "Instead of UTF-8 your shell uses US-ASCII"
-          print "Please add the following to your '~/.bashrc':"
-          print ""
-          print "       export LANG=en_US.UTF-8"
-          print "       export LANGUAGE=en_US.UTF-8"
-          print "       export LC_ALL=en_US.UTF-8"
-          print ""
-          print "You'll have to restart your shell session after updating the file."
-          print "If you are using zshell or another shell, make sure to edit the correct bash file."
-          print "For more information visit this stackoverflow answer:"
-          print "https://stackoverflow.com/a/17031697/445598"
+          print("Your shell environment is not correctly configured")
+          print("Instead of UTF-8 your shell uses US-ASCII")
+          print("Please add the following to your '~/.bashrc':")
+          print("")
+          print("       export LANG=en_US.UTF-8")
+          print("       export LANGUAGE=en_US.UTF-8")
+          print("       export LC_ALL=en_US.UTF-8")
+          print("")
+          print("You'll have to restart your shell session after updating the file.")
+          print("If you are using zshell or another shell, make sure to edit the correct bash file.")
+          print("For more information visit this stackoverflow answer:")
+          print("https://stackoverflow.com/a/17031697/445598")
         end
         print_xcode_path_instructions
         print_xcode_version
@@ -63,11 +63,11 @@ module Gym
         # This error is rather common and should be below the other (a little noisy) output
         case output
         when /Code signing is required for product/
-          print "Seems like Xcode is not happy with the code signing setup"
-          print "Please make sure to check out the raw `xcodebuild` output"
+          print("Seems like Xcode is not happy with the code signing setup")
+          print("Please make sure to check out the raw `xcodebuild` output")
           UI.important(Gym::BuildCommandGenerator.xcodebuild_log_path)
-          print "The very bottom of the file will tell you the raw Xcode error message"
-          print "indicating on why the code signing step failed"
+          print("The very bottom of the file will tell you the raw Xcode error message")
+          print("indicating on why the code signing step failed")
         end
 
         UI.build_failure!("Error building the application - see the log above", error_info: output)
@@ -78,31 +78,31 @@ module Gym
       def handle_package_error(output)
         case output
         when /single\-bundle/
-          print "Your project does not contain a single–bundle application or contains multiple products"
-          print "Please read the documentation provided by Apple: https://developer.apple.com/library/ios/technotes/tn2215/_index.html"
+          print("Your project does not contain a single–bundle application or contains multiple products")
+          print("Please read the documentation provided by Apple: https://developer.apple.com/library/ios/technotes/tn2215/_index.html")
         when /no signing identity matches '(.*)'/
-          print "Could not find code signing identity '#{$1}'"
-          print "Make sure the name of the code signing identity is correct"
-          print "and it matches a locally installed code signing identity"
-          print "You can pass the name of the code signing identity using the"
-          print "`codesigning_identity` option"
+          print("Could not find code signing identity '#{$1}'")
+          print("Make sure the name of the code signing identity is correct")
+          print("and it matches a locally installed code signing identity")
+          print("You can pass the name of the code signing identity using the")
+          print("`codesigning_identity` option")
         when /no provisioning profile matches '(.*)'/
-          print "Could not find provisioning profile with the name '#{$1}'"
-          print "Make sure the name of the provisioning profile is correct"
-          print "and it matches a locally installed profile"
+          print("Could not find provisioning profile with the name '#{$1}'")
+          print("Make sure the name of the provisioning profile is correct")
+          print("and it matches a locally installed profile")
         when /mismatch between specified provisioning profile and signing identity/
-          print "Mismatch between provisioning profile and code signing identity"
-          print "This means, the specified provisioning profile was not created using"
-          print "the specified certificate."
-          print "Run cert and sigh before gym to make sure to have all signing resources ready"
+          print("Mismatch between provisioning profile and code signing identity")
+          print("This means, the specified provisioning profile was not created using")
+          print("the specified certificate.")
+          print("Run cert and sigh before gym to make sure to have all signing resources ready")
         when /requires a provisioning profile/
-          print "No provisioning profile provided"
-          print "Make sure to pass a valid provisioning for each required target"
-          print "Check out the docs on how to fix this: https://docs.fastlane.tools/actions/gym/#export-options"
+          print("No provisioning profile provided")
+          print("Make sure to pass a valid provisioning for each required target")
+          print("Check out the docs on how to fix this: https://docs.fastlane.tools/actions/gym/#export-options")
         # insert more specific code signing errors here
         when /Codesign check fails/
-          print "A general code signing error occurred. Make sure you passed a valid"
-          print "provisioning profile and code signing identity."
+          print("A general code signing error occurred. Make sure you passed a valid")
+          print("provisioning profile and code signing identity.")
         end
         print_xcode_version
         print_full_log_path
@@ -113,17 +113,17 @@ module Gym
       end
 
       def handle_empty_archive
-        print "The generated archive is invalid, this can have various reasons:"
-        print "Usually it's caused by the `Skip Install` option in Xcode, set it to `NO`"
-        print "For more information visit https://developer.apple.com/library/ios/technotes/tn2215/_index.html"
-        print "Also, make sure to have a valid code signing identity and provisioning profile installed"
-        print "Follow this guide to setup code signing https://docs.fastlane.tools/codesigning/GettingStarted/"
-        print "If your intention was only to export an ipa be sure to provide a valid archive at the archive path."
-        print "This error might also happen if your workspace/project file is not in the root directory of your project."
-        print "To workaround that issue, you can wrap your calls to gym with"
-        print "`Dir.chdir('../path/to/dir/containing/proj') do`"
-        print "For an example you can check out"
-        print "https://github.com/artsy/emission-nebula/commit/44fe51a7fea8f7d52f0f77d6c3084827fe5dd59e"
+        print("The generated archive is invalid, this can have various reasons:")
+        print("Usually it's caused by the `Skip Install` option in Xcode, set it to `NO`")
+        print("For more information visit https://developer.apple.com/library/ios/technotes/tn2215/_index.html")
+        print("Also, make sure to have a valid code signing identity and provisioning profile installed")
+        print("Follow this guide to setup code signing https://docs.fastlane.tools/codesigning/GettingStarted/")
+        print("If your intention was only to export an ipa be sure to provide a valid archive at the archive path.")
+        print("This error might also happen if your workspace/project file is not in the root directory of your project.")
+        print("To workaround that issue, you can wrap your calls to gym with")
+        print("`Dir.chdir('../path/to/dir/containing/proj') do`")
+        print("For an example you can check out")
+        print("https://github.com/artsy/emission-nebula/commit/44fe51a7fea8f7d52f0f77d6c3084827fe5dd59e")
         UI.build_failure!("Archive invalid")
       end
 
@@ -131,7 +131,7 @@ module Gym
 
       # Just to make things easier
       def print(text)
-        UI.error text
+        UI.error(text)
       end
 
       def print_full_log_path
@@ -196,25 +196,25 @@ module Gym
 
         xcode_installations_in_default_path = Dir[File.join(default_xcode_path, "Xcode*.app")]
         return unless xcode_installations_in_default_path.count > 1
-        UI.message ""
-        UI.important "Maybe the error shown is caused by using the wrong version of Xcode"
-        UI.important "Found multiple versions of Xcode in '#{default_xcode_path}'"
-        UI.important "Make sure you selected the right version for your project"
-        UI.important "This build process was executed using '#{xcode_path}'"
-        UI.important "If you want to update your Xcode path, either"
-        UI.message ""
+        UI.message("")
+        UI.important("Maybe the error shown is caused by using the wrong version of Xcode")
+        UI.important("Found multiple versions of Xcode in '#{default_xcode_path}'")
+        UI.important("Make sure you selected the right version for your project")
+        UI.important("This build process was executed using '#{xcode_path}'")
+        UI.important("If you want to update your Xcode path, either")
+        UI.message("")
 
-        UI.message "- Specify the Xcode version in your Fastfile"
-        UI.command_output "xcversion(version: \"8.1\") # Selects Xcode 8.1.0"
-        UI.message ""
+        UI.message("- Specify the Xcode version in your Fastfile")
+        UI.command_output("xcversion(version: \"8.1\") # Selects Xcode 8.1.0")
+        UI.message("")
 
-        UI.message "- Specify an absolute path to your Xcode installation in your Fastfile"
-        UI.command_output "xcode_select \"/Applications/Xcode8.app\""
-        UI.message ""
+        UI.message("- Specify an absolute path to your Xcode installation in your Fastfile")
+        UI.command_output("xcode_select \"/Applications/Xcode8.app\"")
+        UI.message("")
 
-        UI.message "- Manually update the path using"
-        UI.command_output "sudo xcode-select -s /Applications/Xcode.app"
-        UI.message ""
+        UI.message("- Manually update the path using")
+        UI.command_output("sudo xcode-select -s /Applications/Xcode.app")
+        UI.message("")
       end
 
       def print_environment_information