brew style --fix --todo

Author: MikeMcQuaid

Library/Homebrew/brew_irb_helpers.rb

@@ -21,7 +21,7 @@ def c(config: nil)
   end
 end
 
-class Symbol
+class Symbol # rubocop:todo Style/OneClassPerFile
   # @!visibility private
   sig { params(args: Integer).returns(Formula) }
   def f(*args)

Library/Homebrew/cache_store.rb

@@ -189,7 +189,7 @@ def dirty?
 # {CacheStore} provides methods to mutate and fetch data from a persistent
 # storage mechanism.
 #
-class CacheStore
+class CacheStore # rubocop:todo Style/OneClassPerFile
   extend T::Helpers
 
   abstract!

Library/Homebrew/cmd/update-report.rb

@@ -370,7 +370,7 @@ def install_from_api_message
 
 require "extend/os/cmd/update-report"
 
-class Reporter
+class Reporter # rubocop:todo Style/OneClassPerFile
   include Utils::Output::Mixin
 
   Report = T.type_alias do
@@ -774,7 +774,7 @@ def diff
   end
 end
 
-class ReporterHub
+class ReporterHub # rubocop:todo Style/OneClassPerFile
   include Utils::Output::Mixin
 
   sig { returns(T::Array[Reporter]) }

Library/Homebrew/compilers.rb

@@ -17,7 +17,7 @@ module CompilerConstants
 end
 
 # Class for checking compiler compatibility for a formula.
-class CompilerFailure
+class CompilerFailure # rubocop:todo Style/OneClassPerFile
   sig { returns(Symbol) }
   attr_reader :type
 
@@ -97,7 +97,7 @@ def gcc_major(version)
 end
 
 # Class for selecting a compiler for a formula.
-class CompilerSelector
+class CompilerSelector # rubocop:todo Style/OneClassPerFile
   include CompilerConstants
 
   class Compiler < T::Struct

Library/Homebrew/dependencies.rb

@@ -42,7 +42,7 @@ def inspect
 end
 
 # A collection of requirements.
-class Requirements < SimpleDelegator
+class Requirements < SimpleDelegator # rubocop:todo Style/OneClassPerFile
   def initialize(*args)
     super(Set.new(args))
   end

Library/Homebrew/dependencies.rbi

@@ -18,7 +18,7 @@ class Dependencies < SimpleDelegator
   def select(&blk); end
 end
 
-class Requirements < SimpleDelegator
+class Requirements < SimpleDelegator # rubocop:todo Style/OneClassPerFile
   include Enumerable
   include Kernel
 

Library/Homebrew/dependency.rb

@@ -297,7 +297,7 @@ def merge_temporality(deps)
 end
 
 # A dependency that's marked as "installed" on macOS
-class UsesFromMacOSDependency < Dependency
+class UsesFromMacOSDependency < Dependency # rubocop:todo Style/OneClassPerFile
   attr_reader :bounds
 
   sig { params(name: String, tags: T::Array[Symbol], bounds: T::Hash[Symbol, Symbol]).void }

Library/Homebrew/description_cache_store.rb

@@ -97,7 +97,7 @@ def select(&block)
 # {CaskDescriptionCacheStore} provides methods to fetch and mutate cask descriptions used
 # by the `brew desc` and `brew search` commands.
 #
-class CaskDescriptionCacheStore < DescriptionCacheStore
+class CaskDescriptionCacheStore < DescriptionCacheStore # rubocop:todo Style/OneClassPerFile
   # If the database is empty `update!` it with all known casks.
   #
   # @return [nil]

Library/Homebrew/download_strategy.rb

@@ -23,7 +23,7 @@ class Mechanize; end
 require "github_packages"
 
 # @abstract Abstract superclass for all download strategies.
-class AbstractDownloadStrategy
+class AbstractDownloadStrategy # rubocop:todo Style/OneClassPerFile
   extend T::Helpers
   include FileUtils
   include Context
@@ -203,7 +203,7 @@ def env
 end
 
 # @abstract Abstract superclass for all download strategies downloading from a version control system.
-class VCSDownloadStrategy < AbstractDownloadStrategy
+class VCSDownloadStrategy < AbstractDownloadStrategy # rubocop:todo Style/OneClassPerFile
   abstract!
 
   sig { override.returns(Pathname) }
@@ -303,7 +303,7 @@ def extract_ref(specs)
 end
 
 # @abstract Abstract superclass for all download strategies downloading a single file.
-class AbstractFileDownloadStrategy < AbstractDownloadStrategy
+class AbstractFileDownloadStrategy < AbstractDownloadStrategy # rubocop:todo Style/OneClassPerFile
   abstract!
 
   # Path for storing an incomplete download while the download is still in progress.
@@ -431,7 +431,7 @@ def parse_basename(url, search_query: true)
 # Strategy for downloading files using `curl`.
 #
 # @api public
-class CurlDownloadStrategy < AbstractFileDownloadStrategy
+class CurlDownloadStrategy < AbstractFileDownloadStrategy # rubocop:todo Style/OneClassPerFile
   include Utils::Curl
 
   # url, basename, time, file_size, content_type, is_redirection
@@ -701,7 +701,7 @@ def curl(*args, print_stdout: true, **options)
 # Strategy for downloading a file using Homebrew's `curl`.
 #
 # @api public
-class HomebrewCurlDownloadStrategy < CurlDownloadStrategy
+class HomebrewCurlDownloadStrategy < CurlDownloadStrategy # rubocop:todo Style/OneClassPerFile
   private
 
   sig {
@@ -726,7 +726,7 @@ def curl_output(*args, **options)
 # Strategy for downloading a file from an GitHub Packages URL.
 #
 # @api public
-class CurlGitHubPackagesDownloadStrategy < CurlDownloadStrategy
+class CurlGitHubPackagesDownloadStrategy < CurlDownloadStrategy # rubocop:todo Style/OneClassPerFile
   sig { params(resolved_basename: String).returns(T.nilable(String)) }
   attr_writer :resolved_basename
 
@@ -760,7 +760,7 @@ def resolve_url_basename_time_file_size(url, timeout: nil)
 # Strategy for downloading a file from an Apache Mirror URL.
 #
 # @api public
-class CurlApacheMirrorDownloadStrategy < CurlDownloadStrategy
+class CurlApacheMirrorDownloadStrategy < CurlDownloadStrategy # rubocop:todo Style/OneClassPerFile
   sig { returns(T::Array[String]) }
   def mirrors
     combined_mirrors
@@ -809,7 +809,7 @@ def apache_mirrors
 # Query parameters on the URL are converted into POST parameters.
 #
 # @api public
-class CurlPostDownloadStrategy < CurlDownloadStrategy
+class CurlPostDownloadStrategy < CurlDownloadStrategy # rubocop:todo Style/OneClassPerFile
   private
 
   sig {
@@ -833,7 +833,7 @@ def _fetch(url:, resolved_url:, timeout:)
 # (Useful for downloading `.jar` files.)
 #
 # @api public
-class NoUnzipCurlDownloadStrategy < CurlDownloadStrategy
+class NoUnzipCurlDownloadStrategy < CurlDownloadStrategy # rubocop:todo Style/OneClassPerFile
   sig { override.params(_block: T.nilable(T.proc.void)).void }
   def stage(&_block)
     UnpackStrategy::Uncompressed.new(cached_location)
@@ -844,7 +844,7 @@ def stage(&_block)
 end
 
 # Strategy for extracting local binary packages.
-class LocalBottleDownloadStrategy < AbstractFileDownloadStrategy
+class LocalBottleDownloadStrategy < AbstractFileDownloadStrategy # rubocop:todo Style/OneClassPerFile
   # TODO: Call `super` here
   # rubocop:disable Lint/MissingSuper
   sig { params(path: Pathname).void }
@@ -862,7 +862,7 @@ def clear_cache
 # Strategy for downloading a Subversion repository.
 #
 # @api public
-class SubversionDownloadStrategy < VCSDownloadStrategy
+class SubversionDownloadStrategy < VCSDownloadStrategy # rubocop:todo Style/OneClassPerFile
   sig { params(url: String, name: String, version: T.nilable(T.any(String, Version)), meta: T.untyped).void }
   def initialize(url, name, version, **meta)
     super
@@ -981,7 +981,7 @@ def clone_repo(timeout: nil)
 # Strategy for downloading a Git repository.
 #
 # @api public
-class GitDownloadStrategy < VCSDownloadStrategy
+class GitDownloadStrategy < VCSDownloadStrategy # rubocop:todo Style/OneClassPerFile
   MINIMUM_COMMIT_HASH_LENGTH = 7
 
   sig { params(url: String, name: String, version: T.nilable(T.any(String, Version)), meta: T.untyped).void }
@@ -1268,7 +1268,7 @@ def configure_sparse_checkout
 # Strategy for downloading a Git repository from GitHub.
 #
 # @api public
-class GitHubGitDownloadStrategy < GitDownloadStrategy
+class GitHubGitDownloadStrategy < GitDownloadStrategy # rubocop:todo Style/OneClassPerFile
   sig { params(url: String, name: String, version: T.nilable(Version), meta: T.untyped).void }
   def initialize(url, name, version, **meta)
     super
@@ -1329,7 +1329,7 @@ def default_branch
 # Strategy for downloading a CVS repository.
 #
 # @api public
-class CVSDownloadStrategy < VCSDownloadStrategy
+class CVSDownloadStrategy < VCSDownloadStrategy # rubocop:todo Style/OneClassPerFile
   sig { params(url: String, name: String, version: T.nilable(T.any(String, Version)), meta: T.untyped).void }
   def initialize(url, name, version, **meta)
     super
@@ -1422,7 +1422,7 @@ def split_url(in_url)
 # Strategy for downloading a Mercurial repository.
 #
 # @api public
-class MercurialDownloadStrategy < VCSDownloadStrategy
+class MercurialDownloadStrategy < VCSDownloadStrategy # rubocop:todo Style/OneClassPerFile
   sig { params(url: String, name: String, version: T.nilable(T.any(String, Version)), meta: T.untyped).void }
   def initialize(url, name, version, **meta)
     super
@@ -1510,7 +1510,7 @@ def current_revision
 # Strategy for downloading a Bazaar repository.
 #
 # @api public
-class BazaarDownloadStrategy < VCSDownloadStrategy
+class BazaarDownloadStrategy < VCSDownloadStrategy # rubocop:todo Style/OneClassPerFile
   sig { params(url: String, name: String, version: T.nilable(T.any(String, Version)), meta: T.untyped).void }
   def initialize(url, name, version, **meta)
     super
@@ -1576,7 +1576,7 @@ def update(timeout: nil)
 # Strategy for downloading a Fossil repository.
 #
 # @api public
-class FossilDownloadStrategy < VCSDownloadStrategy
+class FossilDownloadStrategy < VCSDownloadStrategy # rubocop:todo Style/OneClassPerFile
   sig { params(url: String, name: String, version: T.nilable(T.any(String, Version)), meta: T.untyped).void }
   def initialize(url, name, version, **meta)
     super
@@ -1630,7 +1630,7 @@ def update(timeout: nil)
 end
 
 # Helper class for detecting a download strategy from a URL.
-class DownloadStrategyDetector
+class DownloadStrategyDetector # rubocop:todo Style/OneClassPerFile
   sig {
     params(url: String, using: T.nilable(T.any(Symbol, T::Class[AbstractDownloadStrategy])))
       .returns(T::Class[AbstractDownloadStrategy])

Library/Homebrew/download_strategy.rbi

@@ -10,7 +10,7 @@ end
 
 # This is a third-party implementation.
 # rubocop:disable Style/OptionalBooleanParameter
-class Mechanize::HTTP::ContentDispositionParser
+class Mechanize::HTTP::ContentDispositionParser # rubocop:todo Style/OneClassPerFile
   sig {
     params(content_disposition: String, header: T::Boolean)
       .returns(T.nilable(Mechanize::HTTP::ContentDisposition))