commands_generator.rb in fastlane-2.75.0.beta.20180109010003

- old
+ new

@@ -63,11 +63,11 @@
 
       command :download_all do |c|
         c.syntax = 'fastlane sigh download_all'
         c.description = 'Downloads all valid provisioning profiles'
 
-        c.option '--download_xcode_profiles', 'Only works with `fastlane sigh download_all` command: Also download Xcode managed provisioning profiles'
+        c.option('--download_xcode_profiles', 'Only works with `fastlane sigh download_all` command: Also download Xcode managed provisioning profiles')
 
         FastlaneCore::CommanderGenerator.new.generate(Sigh::Options.available_options, command: c)
 
         c.action do |args, options|
           # Below is some custom code to get an extra flag that's only available
@@ -97,45 +97,45 @@
       end
 
       command :resign do |c|
         c.syntax = 'fastlane sigh resign'
         c.description = 'Resigns an existing ipa file with the given provisioning profile'
-        c.option '-i', '--signing_identity STRING', String, 'The signing identity to use. Must match the one defined in the provisioning profile.'
-        c.option '-x', '--version_number STRING', String, 'Version number to force binary and all nested binaries to use. Changes both CFBundleShortVersionString and CFBundleVersion.'
-        c.option '-p', '--provisioning_profile PATH', String, '(or BUNDLE_ID=PATH) The path to the provisioning profile which should be used. '\
+        c.option('-i', '--signing_identity STRING', String, 'The signing identity to use. Must match the one defined in the provisioning profile.')
+        c.option('-x', '--version_number STRING', String, 'Version number to force binary and all nested binaries to use. Changes both CFBundleShortVersionString and CFBundleVersion.')
+        c.option('-p', '--provisioning_profile PATH', String, '(or BUNDLE_ID=PATH) The path to the provisioning profile which should be used. '\
                  'Can be provided multiple times if the application contains nested applications and app extensions, which need their own provisioning profile. '\
                  'The path may be prefixed with a identifier in order to determine which provisioning profile should be used on which app.',
-                 &multiple_values_option_proc(c, "provisioning_profile", &proc { |value| value.split('=', 2) })
-        c.option '-d', '--display_name STRING', String, 'Display name to use'
-        c.option '-e', '--entitlements PATH', String, 'The path to the entitlements file to use.'
-        c.option '--short_version STRING', String, 'Short version string to force binary and all nested binaries to use (CFBundleShortVersionString).'
-        c.option '--bundle_version STRING', String, 'Bundle version to force binary and all nested binaries to use (CFBundleVersion).'
-        c.option '--use_app_entitlements', 'Extract app bundle codesigning entitlements and combine with entitlements from new provisionin profile.'
-        c.option '-g', '--new_bundle_id STRING', String, 'New application bundle ID (CFBundleIdentifier)'
-        c.option '--keychain_path STRING', String, 'Path to the keychain that /usr/bin/codesign should use'
+                 &multiple_values_option_proc(c, "provisioning_profile", &proc { |value| value.split('=', 2) }))
+        c.option('-d', '--display_name STRING', String, 'Display name to use')
+        c.option('-e', '--entitlements PATH', String, 'The path to the entitlements file to use.')
+        c.option('--short_version STRING', String, 'Short version string to force binary and all nested binaries to use (CFBundleShortVersionString).')
+        c.option('--bundle_version STRING', String, 'Bundle version to force binary and all nested binaries to use (CFBundleVersion).')
+        c.option('--use_app_entitlements', 'Extract app bundle codesigning entitlements and combine with entitlements from new provisionin profile.')
+        c.option('-g', '--new_bundle_id STRING', String, 'New application bundle ID (CFBundleIdentifier)')
+        c.option('--keychain_path STRING', String, 'Path to the keychain that /usr/bin/codesign should use')
 
         c.action do |args, options|
           Sigh::Resign.new.run(options, args)
         end
       end
 
       command :manage do |c|
         c.syntax = 'fastlane sigh manage'
         c.description = 'Manage installed provisioning profiles on your system.'
 
-        c.option '-f', '--force', 'Force remove all expired provisioning profiles. Required on CI.'
-        c.option '-e', '--clean_expired', 'Remove all expired provisioning profiles.'
+        c.option('-f', '--force', 'Force remove all expired provisioning profiles. Required on CI.')
+        c.option('-e', '--clean_expired', 'Remove all expired provisioning profiles.')
 
-        c.option '-p', '--clean_pattern STRING', String, 'Remove any provisioning profiles that matches the regular expression.'
-        c.example 'Remove all "iOS Team Provisioning" provisioning profiles', 'fastlane sigh manage -p "iOS\ ?Team Provisioning Profile"'
+        c.option('-p', '--clean_pattern STRING', String, 'Remove any provisioning profiles that matches the regular expression.')
+        c.example('Remove all "iOS Team Provisioning" provisioning profiles', 'fastlane sigh manage -p "iOS\ ?Team Provisioning Profile"')
 
         c.action do |args, options|
           Sigh::LocalManage.start(options, args)
         end
       end
 
-      default_command :renew
+      default_command(:renew)
 
       run!
     end
 
     def multiple_values_option_proc(command, name)
@@ -143,10 +143,10 @@
         value = yield(value) if block_given?
         option = command.proxy_options.find { |opt| opt[0] == name } || []
         values = option[1] || []
         values << value
 
-        command.proxy_options.delete option
+        command.proxy_options.delete(option)
         command.proxy_options << [name, values]
       end
     end
   end
 end