Remove invalid HOMEBREW_CASK_OPTS options

Library/Homebrew/cask/cmd/reinstall.rb

@@ -6,31 +6,28 @@ module Cask
       def run
         self.class.reinstall_casks(
           *casks,
-          binaries: binaries?,
+          binaries:       binaries?,
-          verbose: verbose?,
+          verbose:        verbose?,
-          force: force?,
+          force:          force?,
           skip_cask_deps: skip_cask_deps?,
-          require_sha: require_sha?,
+          require_sha:    require_sha?,
-          quarantine: quarantine?,
+          quarantine:     quarantine?,
         )
       end
 
       def self.reinstall_casks(
         *casks,
+        verbose: false,
+        force: false,
+        skip_cask_deps: false,
         binaries: nil,
-        verbose: nil,
-        force: nil,
-        skip_cask_deps: nil,
         require_sha: nil,
         quarantine: nil
       )
         # TODO: Handle this in `CLI::Parser`.
-        binaries       = Homebrew::EnvConfig.cask_opts_binaries?       if binaries.nil?
+        binaries = Homebrew::EnvConfig.cask_opts_binaries? if binaries.nil?
-        force          = Homebrew::EnvConfig.cask_opts_force?          if force.nil?
+        quarantine = Homebrew::EnvConfig.cask_opts_quarantine? if quarantine.nil?
-        quarantine     = Homebrew::EnvConfig.cask_opts_quarantine?     if quarantine.nil?
+        require_sha = Homebrew::EnvConfig.cask_opts_require_sha? if require_sha.nil?
-        require_sha    = Homebrew::EnvConfig.cask_opts_require_sha?    if require_sha.nil?
-        skip_cask_deps = Homebrew::EnvConfig.cask_opts_skip_cask_deps? if skip_cask_deps.nil?
-        verbose        = Homebrew::EnvConfig.cask_opts_verbose?        if verbose.nil?
 
         casks.each do |cask|
           Installer.new(cask,

Library/Homebrew/cask/cmd/uninstall.rb

@@ -14,16 +14,13 @@ module Cask
         self.class.uninstall_casks(
           *casks,
           binaries: binaries?,
-          verbose: verbose?,
+          verbose:  verbose?,
-          force: force?
+          force:    force?,
         )
       end
 
-      def self.uninstall_casks(*casks, binaries: nil, verbose: nil, force: nil)
+      def self.uninstall_casks(*casks, verbose: false, force: false, binaries: nil)
-        # TODO: Handle this in `CLI::Parser`.
         binaries = Homebrew::EnvConfig.cask_opts_binaries? if binaries.nil?
-        force    = Homebrew::EnvConfig.cask_opts_force?    if force.nil?
-        verbose  = Homebrew::EnvConfig.cask_opts_verbose?  if verbose.nil?
 
         casks.each do |cask|
           odebug "Uninstalling Cask #{cask}"

Library/Homebrew/cask/cmd/upgrade.rb

@@ -33,23 +33,19 @@ module Cask
 
       def self.upgrade_casks(
         *casks,
-        force: nil,
+        force: false,
-        greedy: nil,
+        greedy: false,
-        dry_run: nil,
+        dry_run: false,
+        skip_cask_deps: false,
+        verbose: false,
         binaries: nil,
-        skip_cask_deps: nil,
-        verbose: nil,
         quarantine: nil,
         require_sha: nil
       )
         # TODO: Handle this in `CLI::Parser`.
-        binaries    = Homebrew::EnvConfig.cask_opts_binaries?    if binaries.nil?
+        binaries = Homebrew::EnvConfig.cask_opts_binaries? if binaries.nil?
-        dry_run     = Homebrew::EnvConfig.cask_opts_dry_run?     if dry_run.nil?
+        quarantine = Homebrew::EnvConfig.cask_opts_quarantine? if quarantine.nil?
-        force       = Homebrew::EnvConfig.cask_opts_force?       if force.nil?
-        greedy      = Homebrew::EnvConfig.cask_opts_greedy?      if greedy.nil?
-        quarantine  = Homebrew::EnvConfig.cask_opts_quarantine?  if quarantine.nil?
         require_sha = Homebrew::EnvConfig.cask_opts_require_sha? if require_sha.nil?
-        verbose     = Homebrew::EnvConfig.cask_opts_verbose?     if verbose.nil?
 
         outdated_casks = if casks.empty?
           Caskroom.casks.select do |cask|

Library/Homebrew/env_config.rb

@@ -332,18 +332,6 @@ module Homebrew
       true
     end
 
-    def cask_opts_dry_run?
-      cask_opts.include?("--dry-run")
-    end
-
-    def cask_opts_force?
-      cask_opts.include?("--force")
-    end
-
-    def cask_opts_greedy?
-      cask_opts.include?("--greedy")
-    end
-
     def cask_opts_quarantine?
       cask_opts.reverse_each do |opt|
         return true if opt == "--quarantine"
@@ -356,13 +344,5 @@ module Homebrew
     def cask_opts_require_sha?
       cask_opts.include?("--require-sha")
     end
-
-    def cask_opts_skip_cask_deps?
-      cask_opts.include?("--skip-cask-deps")
-    end
-
-    def cask_opts_verbose?
-      cask_opts.include?("--verbose")
-    end
   end
 end