Apply suggestions from code review

Co-authored-by: Douglas Eichelberger <697964+dduugg@users.noreply.github.com>
Homebrew · Jan 14, 2025 · 43f989e · 43f989e
1 parent 6b645b5
commit 43f989e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 11 deletions.
diff --git a/Library/Homebrew/extend/os/linux/cask/config.rb b/Library/Homebrew/extend/os/linux/cask/config.rb
@@ -7,19 +7,15 @@ module OS
   module Linux
     module Cask
       module Config
-        extend T::Helpers
-
-        requires_ancestor { ::Cask::Config }
-
         module ClassMethods
           DEFAULT_DIRS = T.let({
             vst_plugindir:  "~/.vst",
             vst3_plugindir: "~/.vst3",
             fontdir:        "#{ENV.fetch("XDG_DATA_HOME", "~/.local/share")}/fonts",
-          }.freeze, T::Hash[Symbol, T.nilable(String)])
+          }.freeze, T::Hash[Symbol, String])
 
-          sig { returns(T::Hash[Symbol, T.untyped]) }
-          def self.defaults
+          sig { returns(T::Hash[Symbol, String]) }
+          def defaults
             {
               languages: LazyObject.new { Linux.languages },
             }.merge(DEFAULT_DIRS).freeze
@@ -31,4 +27,3 @@ def self.defaults
 end
 
 Cask::Config.singleton_class.prepend(OS::Linux::Cask::Config::ClassMethods)
-Cask::Config.prepend(OS::Linux::Cask::Config)
diff --git a/Library/Homebrew/extend/os/linux/cask/quarantine.rb b/Library/Homebrew/extend/os/linux/cask/quarantine.rb
@@ -10,9 +10,7 @@ module Quarantine
         requires_ancestor { ::Cask::Quarantine }
 
         sig { returns(Symbol) }
-        def self.check_quarantine_support
-          :linux
-        end
+        def self.check_quarantine_support = :linux
 
         sig { returns(T::Boolean) }
         def self.available? = false