Class: RuboCop::Cop::Modularization::ClassMethodsAsPublicApis
- Inherits:
-
Base
- Object
- Base
- RuboCop::Cop::Modularization::ClassMethodsAsPublicApis
- Extended by:
- T::Sig
- Defined in:
- lib/rubocop/cop/modularization/class_methods_as_public_apis.rb
Overview
This cop states that public API should live on class methods, which are more easily statically analyzable, searchable, and typically hold less state.
Instance Method Summary collapse
Instance Method Details
#on_def(node) ⇒ Object
45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 |
# File 'lib/rubocop/cop/modularization/class_methods_as_public_apis.rb', line 45 def on_def(node) # This cop only applies for ruby files in `app/public` return if !processed_source.file_path.include?('app/public') # Looked at https://www.rubydoc.info/gems/rubocop/RuboCop/Cop/Lint/MissingSuper source code as inspiration for htis part. class_node = node.each_ancestor(:class).first module_node = node.each_ancestor(:module).first parent_class = class_node&.parent_class || module_node&.parent acceptable_parent_classes = cop_config['AcceptableParentClasses'] || [] # Used this PR as inspiration to check if we're within a `class << self` block uses_implicit_static_methods = node.each_ancestor(:sclass).first&.identifier&.source == 'self' class_is_allowed_to_have_instance_methods = acceptable_parent_classes.include?(parent_class&.const_name) return if uses_implicit_static_methods || class_is_allowed_to_have_instance_methods add_offense( node.source_range, message: format( 'Top-level files in the public/ folder may only define class methods.' ) ) end |
#support_autocorrect? ⇒ Boolean
40 41 42 |
# File 'lib/rubocop/cop/modularization/class_methods_as_public_apis.rb', line 40 def support_autocorrect? false end |