Use exclude_from_registry instead

Library/Homebrew/rubocops/bottle.rb

@@ -9,8 +9,7 @@ module RuboCop
       # This cop audits the `bottle` block in formulae.
       #
       # @api private
-      class BottleFormat < Base
+      class BottleFormat < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -57,8 +56,7 @@ module RuboCop
       # This cop audits the indentation of the bottle tags in the `bottle` block in formulae.
       #
       # @api private
-      class BottleTagIndentation < Base
+      class BottleTagIndentation < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -94,8 +92,7 @@ module RuboCop
       # This cop audits the indentation of the sha256 digests in the`bottle` block in formulae.
       #
       # @api private
-      class BottleDigestIndentation < Base
+      class BottleDigestIndentation < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -131,8 +128,7 @@ module RuboCop
       # This cop audits the order of the `bottle` block in formulae.
       #
       # @api private
-      class BottleOrder < Base
+      class BottleOrder < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)

Library/Homebrew/rubocops/caveats.rb

@@ -24,9 +24,7 @@ module RuboCop
       #   end
       #
       # @api private
-      class Caveats < Base
+      class Caveats < FormulaCop
-        include FormulaCop
-
         def audit_formula(_node, _class_node, _parent_class_node, _body_node)
           caveats_strings.each do |n|
             if regex_match_group(n, /\bsetuid\b/i)

Library/Homebrew/rubocops/checksum.rb

@@ -9,9 +9,7 @@ module RuboCop
       # This cop makes sure that deprecated checksums are not used.
       #
       # @api private
-      class Checksum < Base
+      class Checksum < FormulaCop
-        include FormulaCop
-
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           return if body_node.nil?
 
@@ -47,8 +45,7 @@ module RuboCop
       # This cop makes sure that checksum strings are lowercase.
       #
       # @api private
-      class ChecksumCase < Base
+      class ChecksumCase < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)

Library/Homebrew/rubocops/class.rb

@@ -9,8 +9,7 @@ module RuboCop
       # This cop makes sure that {Formula} is used as superclass.
       #
       # @api private
-      class ClassName < Base
+      class ClassName < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         DEPRECATED_CLASSES = %w[
@@ -32,8 +31,7 @@ module RuboCop
       # This cop makes sure that a `test` block contains a proper test.
       #
       # @api private
-      class Test < Base
+      class Test < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -76,8 +74,7 @@ module RuboCop
       # This cop makes sure that a `test` block exists.
       #
       # @api private
-      class TestPresent < Base
+      class TestPresent < FormulaCop
-        include FormulaCop
         def audit_formula(_node, class_node, _parent_class_node, body_node)
           return if find_block(body_node, :test)
 

Library/Homebrew/rubocops/components_order.rb

@@ -11,8 +11,7 @@ module RuboCop
       #
       # - `component_precedence_list` has component hierarchy in a nested list
       #   where each sub array contains components' details which are at same precedence level
-      class ComponentsOrder < Base
+      class ComponentsOrder < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)

Library/Homebrew/rubocops/components_redundancy.rb

@@ -14,9 +14,7 @@ module RuboCop
       # - `stable do` should not be present without a `head` spec
       #
       # @api private
-      class ComponentsRedundancy < Base
+      class ComponentsRedundancy < FormulaCop
-        include FormulaCop
-
         HEAD_MSG = "`head` and `head do` should not be simultaneously present"
         BOTTLE_MSG = "`bottle :modifier` and `bottle do` should not be simultaneously present"
         STABLE_MSG = "`stable do` should not be present without a `head` spec"

Library/Homebrew/rubocops/conflicts.rb

@@ -7,8 +7,7 @@ module RuboCop
   module Cop
     module FormulaAudit
       # This cop audits versioned formulae for `conflicts_with`.
-      class Conflicts < Base
+      class Conflicts < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         MSG = "Versioned formulae should not use `conflicts_with`. " \

Library/Homebrew/rubocops/dependency_order.rb

@@ -10,8 +10,7 @@ module RuboCop
       #
       # precedence order:
       # build-time > test > normal > recommended > optional
-      class DependencyOrder < Base
+      class DependencyOrder < FormulaCop
-        include FormulaCop
         extend T::Sig
         extend AutoCorrector
 

Library/Homebrew/rubocops/deprecate_disable.rb

@@ -7,8 +7,7 @@ module RuboCop
   module Cop
     module FormulaAudit
       # This cop audits `deprecate!` and `disable!` dates.
-      class DeprecateDisableDate < Base
+      class DeprecateDisableDate < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -35,8 +34,7 @@ module RuboCop
       end
 
       # This cop audits `deprecate!` and `disable!` reasons.
-      class DeprecateDisableReason < Base
+      class DeprecateDisableReason < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         PUNCTUATION_MARKS = %w[. ! ?].freeze

Library/Homebrew/rubocops/desc.rb

@@ -9,8 +9,7 @@ module RuboCop
     module FormulaAudit
       # This cop audits `desc` in formulae.
       # See the {DescHelper} module for details of the checks.
-      class Desc < Base
+      class Desc < FormulaCop
-        include FormulaCop
         include DescHelper
         extend AutoCorrector
 

Library/Homebrew/rubocops/extend/formula_cop.rb

@@ -5,14 +5,18 @@ require "rubocops/shared/helper_functions"
 
 module RuboCop
   module Cop
-    # Mixin for all formula cops.
+    # Abstract base class for all formula cops.
     #
     # @api private
-    module FormulaCop
+    class FormulaCop < Cop
-      extend RuboCop::AST::NodePattern::Macros
+      extend T::Sig
+      extend T::Helpers
       include RangeHelp
       include HelperFunctions
 
+      abstract!
+      exclude_from_registry
+
       attr_accessor :file_path
 
       @registry = Cop.registry
@@ -29,9 +33,16 @@ module RuboCop
         audit_formula(node, class_node, parent_class_node, @body)
       end
 
-      def audit_formula(node, class_node, parent_class_node, body_node)
+      sig {
-        raise NotImplementedError, "Subclasses must implement this method."
+        abstract
-      end
+          .params(
+            node:              RuboCop::AST::ClassNode,
+            class_node:        RuboCop::AST::ConstNode,
+            parent_class_node: RuboCop::AST::ConstNode,
+            body_node:         RuboCop::AST::Node,
+          ).void
+      }
+      def audit_formula(node, class_node, parent_class_node, body_node); end
 
       # Yields to block when there is a match.
       #

Library/Homebrew/rubocops/files.rb

@@ -9,9 +9,7 @@ module RuboCop
       # This cop makes sure that a formula's file permissions are correct.
       #
       # @api private
-      class Files < Base
+      class Files < FormulaCop
-        include FormulaCop
-
         def audit_formula(node, _class_node, _parent_class_node, _body_node)
           return unless file_path
 

Library/Homebrew/rubocops/homepage.rb

@@ -7,8 +7,7 @@ module RuboCop
   module Cop
     module FormulaAudit
       # This cop audits the `homepage` URL in formulae.
-      class Homepage < Base
+      class Homepage < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, class_node, _parent_class_node, body_node)

Library/Homebrew/rubocops/keg_only.rb

@@ -9,8 +9,7 @@ module RuboCop
       # This cop makes sure that a `keg_only` reason has the correct format.
       #
       # @api private
-      class KegOnly < Base
+      class KegOnly < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)

Library/Homebrew/rubocops/lines.rb

@@ -11,8 +11,7 @@ module RuboCop
       # This cop checks for various miscellaneous Homebrew coding styles.
       #
       # @api private
-      class Lines < Base
+      class Lines < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, _body_node)
           [:automake, :ant, :autoconf, :emacs, :expat, :libtool, :mysql, :perl,
            :postgresql, :python, :python3, :rbenv, :ruby].each do |dependency|
@@ -35,8 +34,7 @@ module RuboCop
       # This cop makes sure that a space is used for class inheritance.
       #
       # @api private
-      class ClassInheritance < Base
+      class ClassInheritance < FormulaCop
-        include FormulaCop
         def audit_formula(_node, class_node, parent_class_node, _body_node)
           begin_pos = start_column(parent_class_node)
           end_pos = end_column(class_node)
@@ -50,8 +48,7 @@ module RuboCop
       # This cop makes sure that template comments are removed.
       #
       # @api private
-      class Comments < Base
+      class Comments < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, _body_node)
           audit_comments do |comment|
             [
@@ -92,8 +89,7 @@ module RuboCop
       # This cop makes sure that idiomatic `assert_*` statements are used.
       #
       # @api private
-      class AssertStatements < Base
+      class AssertStatements < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           return if body_node.nil?
 
@@ -120,8 +116,7 @@ module RuboCop
       # This cop makes sure that `option`s are used idiomatically.
       #
       # @api private
-      class OptionDeclarations < Base
+      class OptionDeclarations < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           return if body_node.nil?
 
@@ -206,8 +201,7 @@ module RuboCop
       # This cop makes sure that formulae depend on `open-mpi` instead of `mpich`.
       #
       # @api private
-      class MpiCheck < Base
+      class MpiCheck < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -229,8 +223,7 @@ module RuboCop
       # or run-time.
       #
       # @api private
-      class PyoxidizerCheck < Base
+      class PyoxidizerCheck < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           return if body_node.nil?
 
@@ -257,8 +250,7 @@ module RuboCop
       # This cop makes sure that the safe versions of `popen_*` calls are used.
       #
       # @api private
-      class SafePopenCommands < Base
+      class SafePopenCommands < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -289,8 +281,7 @@ module RuboCop
       # This cop makes sure that environment variables are passed correctly to `popen_*` calls.
       #
       # @api private
-      class ShellVariables < Base
+      class ShellVariables < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -322,8 +313,7 @@ module RuboCop
       # This cop makes sure that `license` has the correct format.
       #
       # @api private
-      class LicenseArrays < Base
+      class LicenseArrays < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -344,8 +334,7 @@ module RuboCop
       # This cop makes sure that nested `license` declarations are split onto multiple lines.
       #
       # @api private
-      class Licenses < Base
+      class Licenses < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           return if body_node.nil?
 
@@ -368,8 +357,7 @@ module RuboCop
       # This cop makes sure that Python versions are consistent.
       #
       # @api private
-      class PythonVersions < Base
+      class PythonVersions < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -407,8 +395,7 @@ module RuboCop
       # This cop makes sure that OS conditionals are consistent.
       #
       # @api private
-      class OnSystemConditionals < Base
+      class OnSystemConditionals < FormulaCop
-        include FormulaCop
         include OnSystemConditionalsHelper
         extend AutoCorrector
 
@@ -449,8 +436,7 @@ module RuboCop
       # This cop makes sure that the `generate_completions_from_executable` DSL is used.
       #
       # @api private
-      class GenerateCompletionsDSL < Base
+      class GenerateCompletionsDSL < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -533,8 +519,7 @@ module RuboCop
       # a single, combined call for all shells.
       #
       # @api private
-      class SingleGenerateCompletionsDSLCall < Base
+      class SingleGenerateCompletionsDSLCall < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -596,8 +581,7 @@ module RuboCop
       # This cop checks for other miscellaneous style violations.
       #
       # @api private
-      class Miscellaneous < Base
+      class Miscellaneous < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           return if body_node.nil?
 
@@ -868,8 +852,7 @@ module RuboCop
       # This cop makes sure that no build-time checks are performed.
       #
       # @api private
-      class MakeCheck < Base
+      class MakeCheck < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           return if formula_tap != "homebrew-core"
 
@@ -893,8 +876,7 @@ module RuboCop
       end
 
       # This cop ensures that new formulae depending on removed Requirements are not used
-      class Requirements < Base
+      class Requirements < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, _body_node)
           problem "Formulae should depend on a versioned `openjdk` instead of :java" if depends_on? :java
           problem "Formulae should depend on specific X libraries instead of :x11" if depends_on? :x11

Library/Homebrew/rubocops/livecheck.rb

@@ -10,8 +10,7 @@ module RuboCop
       # skipped formulae.
       #
       # @api private
-      class LivecheckSkip < Base
+      class LivecheckSkip < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -43,8 +42,7 @@ module RuboCop
       # This cop ensures that a `url` is specified in the `livecheck` block.
       #
       # @api private
-      class LivecheckUrlProvided < Base
+      class LivecheckUrlProvided < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           livecheck_node = find_block(body_node, :livecheck)
           return if livecheck_node.blank?
@@ -68,8 +66,7 @@ module RuboCop
       # is used when the livecheck `url` is identical to one of these formula URLs.
       #
       # @api private
-      class LivecheckUrlSymbol < Base
+      class LivecheckUrlSymbol < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -126,8 +123,7 @@ module RuboCop
       # This cop ensures that the `regex` call in the `livecheck` block uses parentheses.
       #
       # @api private
-      class LivecheckRegexParentheses < Base
+      class LivecheckRegexParentheses < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -154,8 +150,7 @@ module RuboCop
       # `\.tgz`, `\.tar.gz` and variants.
       #
       # @api private
-      class LivecheckRegexExtension < Base
+      class LivecheckRegexExtension < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         TAR_PATTERN = /\\?\.t(ar|(g|l|x)z$|[bz2]{2,4}$)(\\?\.((g|l|x)z)|[bz2]{2,4}|Z)?$/i.freeze
@@ -188,8 +183,7 @@ module RuboCop
       # in the `livecheck` block.
       #
       # @api private
-      class LivecheckRegexIfPageMatch < Base
+      class LivecheckRegexIfPageMatch < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           livecheck_node = find_block(body_node, :livecheck)
           return if livecheck_node.blank?
@@ -215,8 +209,7 @@ module RuboCop
       # unless sensitivity is explicitly required for proper matching.
       #
       # @api private
-      class LivecheckRegexCaseInsensitive < Base
+      class LivecheckRegexCaseInsensitive < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         MSG = "Regexes should be case-insensitive unless sensitivity is explicitly required for proper matching."

Library/Homebrew/rubocops/options.rb

@@ -7,8 +7,7 @@ module RuboCop
   module Cop
     module FormulaAudit
       # This cop audits `option`s in formulae.
-      class Options < Base
+      class Options < FormulaCop
-        include FormulaCop
         DEPRECATION_MSG = "macOS has been 64-bit only since 10.6 so 32-bit options are deprecated."
         UNI_DEPRECATION_MSG = "macOS has been 64-bit only since 10.6 so universal options are deprecated."
 

Library/Homebrew/rubocops/patches.rb

@@ -8,8 +8,7 @@ module RuboCop
     module FormulaAudit
       # This cop audits `patch`es in formulae.
       # TODO: Many of these could be auto-corrected.
-      class Patches < Base
+      class Patches < FormulaCop
-        include FormulaCop
         extend T::Sig
         extend AutoCorrector
 

Library/Homebrew/rubocops/text.rb

@@ -9,8 +9,7 @@ module RuboCop
       # This cop checks for various problems in a formula's source code.
       #
       # @api private
-      class Text < Base
+      class Text < FormulaCop
-        include FormulaCop
         extend AutoCorrector
 
         def audit_formula(node, _class_node, _parent_class_node, body_node)
@@ -107,8 +106,7 @@ module RuboCop
       # This cop contains stricter checks for various problems in a formula's source code.
       #
       # @api private
-      class Text < Base
+      class Text < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           return if body_node.nil?
 

Library/Homebrew/rubocops/urls.rb

@@ -9,9 +9,7 @@ module RuboCop
       # This cop audits `url`s and `mirror`s in formulae.
       #
       # @api private
-      class Urls < Base
+      class Urls < FormulaCop
-        include FormulaCop
-
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           return if body_node.nil?
 
@@ -262,8 +260,7 @@ module RuboCop
       # This cop makes sure that the correct format for PyPI URLs is used.
       #
       # @api private
-      class PyPiUrls < Base
+      class PyPiUrls < FormulaCop
-        include FormulaCop
         extend T::Sig
 
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
@@ -297,8 +294,7 @@ module RuboCop
       # This cop makes sure that git URLs have a `revision`.
       #
       # @api private
-      class GitUrls < Base
+      class GitUrls < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           return if body_node.nil?
           return unless formula_tap == "homebrew-core"
@@ -322,8 +318,7 @@ module RuboCop
       # This cop makes sure that git URLs have a `tag`.
       #
       # @api private
-      class GitUrls < Base
+      class GitUrls < FormulaCop
-        include FormulaCop
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           return if body_node.nil?
           return unless formula_tap == "homebrew-core"

Library/Homebrew/rubocops/uses_from_macos.rb

@@ -7,9 +7,7 @@ module RuboCop
   module Cop
     module FormulaAudit
       # This cop audits formulae that are keg-only because they are provided by macos.
-      class ProvidedByMacos < Base
+      class ProvidedByMacos < FormulaCop
-        include FormulaCop
-
         PROVIDED_BY_MACOS_FORMULAE = %w[
           apr
           bc
@@ -75,9 +73,7 @@ module RuboCop
       end
 
       # This cop audits `uses_from_macos` dependencies in formulae.
-      class UsesFromMacos < Base
+      class UsesFromMacos < FormulaCop
-        include FormulaCop
-
         # These formulae aren't `keg_only :provided_by_macos` but are provided by
         # macOS (or very similarly, e.g. OpenSSL where system provides LibreSSL).
         # TODO: consider making some of these keg-only.

Library/Homebrew/rubocops/version.rb

@@ -9,9 +9,7 @@ module RuboCop
       # This cop makes sure that a `version` is in the correct format.
       #
       # @api private
-      class Version < Base
+      class Version < FormulaCop
-        include FormulaCop
-
         def audit_formula(_node, _class_node, _parent_class_node, body_node)
           version_node = find_node_method_by_name(body_node, :version)
           return unless version_node

Library/Homebrew/sorbet/rbi/hidden-definitions/hidden.rbi

@@ -6963,7 +6963,7 @@ class RuboCop::Cop::FormulaAuditStrict::Text
   def share_path_starts_with(param0, param1); end
 end
 
-module RuboCop::Cop::FormulaCop
+class RuboCop::Cop::FormulaCop
   def dependency_name_hash_match?(param0, param1); end
 
   def dependency_type_hash_match?(param0, param1); end