Merge pull request #8390 from MLH-Fellowship/deprecate-cask

cask: Add TODOs to deprecate integrated cask commands

Library/Homebrew/cask/cmd.rb

@@ -47,6 +47,16 @@ module Cask
       "dr"       => "doctor",
     }.freeze
 
+    DEPRECATED_COMMANDS = {
+      Cmd::Cache     => "brew --cache --cask",
+      Cmd::Doctor    => "brew doctor --verbose",
+      Cmd::Home      => "brew home",
+      Cmd::List      => "brew list --cask",
+      Cmd::Outdated  => "brew outdated --cask",
+      Cmd::Reinstall => "brew reinstall",
+      Cmd::Upgrade   => "brew upgrade --cask",
+    }.freeze
+
     def self.description
       max_command_len = Cmd.commands.map(&:length).max
 
@@ -212,6 +222,11 @@ module Cask
                       detect_external_command(*argv) ||
                       [args.remaining.empty? ? NullCommand : UnknownSubcommand.new(args.remaining.first), argv]
 
+      # TODO: enable for next major/minor release
+      # if (replacement = DEPRECATED_COMMANDS[command])
+      #   odeprecated "brew cask #{command.command_name}", replacement
+      # end
+
       if args.help?
         puts command.help
       else

Library/Homebrew/cask/cmd/--cache.rb

@@ -18,9 +18,6 @@ module Cask
       end
 
       def run
-        # TODO: enable for next major/minor release
-        # odeprecated "brew cask --cache", "brew --cache --cask"
-
         casks.each do |cask|
           puts self.class.cached_location(cask)
         end

Library/Homebrew/cask/cmd/home.rb

@@ -8,9 +8,6 @@ module Cask
       end
 
       def run
-        # TODO: enable for next major/minor release
-        # odeprecated "brew cask home", "brew home"
-
         if casks.none?
           odebug "Opening project homepage"
           self.class.open_url "https://brew.sh/"