Skip to content

Commit 43f989e

Browse files
SMillerDevdduugg
andcommitted
Apply suggestions from code review
Co-authored-by: Douglas Eichelberger <[email protected]>
1 parent 6b645b5 commit 43f989e

File tree

2 files changed

+4
-11
lines changed

2 files changed

+4
-11
lines changed

Library/Homebrew/extend/os/linux/cask/config.rb

+3-8
Original file line numberDiff line numberDiff line change
@@ -7,19 +7,15 @@ module OS
77
module Linux
88
module Cask
99
module Config
10-
extend T::Helpers
11-
12-
requires_ancestor { ::Cask::Config }
13-
1410
module ClassMethods
1511
DEFAULT_DIRS = T.let({
1612
vst_plugindir: "~/.vst",
1713
vst3_plugindir: "~/.vst3",
1814
fontdir: "#{ENV.fetch("XDG_DATA_HOME", "~/.local/share")}/fonts",
19-
}.freeze, T::Hash[Symbol, T.nilable(String)])
15+
}.freeze, T::Hash[Symbol, String])
2016

21-
sig { returns(T::Hash[Symbol, T.untyped]) }
22-
def self.defaults
17+
sig { returns(T::Hash[Symbol, String]) }
18+
def defaults
2319
{
2420
languages: LazyObject.new { Linux.languages },
2521
}.merge(DEFAULT_DIRS).freeze
@@ -31,4 +27,3 @@ def self.defaults
3127
end
3228

3329
Cask::Config.singleton_class.prepend(OS::Linux::Cask::Config::ClassMethods)
34-
Cask::Config.prepend(OS::Linux::Cask::Config)

Library/Homebrew/extend/os/linux/cask/quarantine.rb

+1-3
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,7 @@ module Quarantine
1010
requires_ancestor { ::Cask::Quarantine }
1111

1212
sig { returns(Symbol) }
13-
def self.check_quarantine_support
14-
:linux
15-
end
13+
def self.check_quarantine_support = :linux
1614

1715
sig { returns(T::Boolean) }
1816
def self.available? = false

0 commit comments

Comments
 (0)