RubyGems - reek - Versions diffs - 6.1.0 → 6.2.0 - Mend

reek 6.1.0 → 6.2.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (54) hide show

checksums.yaml +4 -4
data/.github/dependabot.yml +4 -0
data/.github/workflows/ruby.yml +7 -7
data/.rubocop.yml +2 -10
data/CHANGELOG.md +69 -0
data/CONTRIBUTING.md +7 -10
data/Gemfile +7 -6
data/README.md +28 -28
data/bin/code_climate_reek +54 -5
data/lib/reek/ast/sexp_extensions/arguments.rb +9 -0
data/lib/reek/ast/sexp_extensions/send.rb +21 -6
data/lib/reek/cli/command/todo_list_command.rb +2 -2
data/lib/reek/cli/options.rb +5 -5
data/lib/reek/{report/code_climate → code_climate}/code_climate_configuration.rb +1 -1
data/lib/reek/{report/code_climate → code_climate}/code_climate_configuration.yml +41 -41
data/lib/reek/{report/code_climate → code_climate}/code_climate_fingerprint.rb +2 -2
data/lib/reek/{report/code_climate → code_climate}/code_climate_formatter.rb +1 -1
data/lib/reek/{report/code_climate → code_climate}/code_climate_report.rb +3 -3
data/lib/reek/code_comment.rb +3 -3
data/lib/reek/configuration/app_configuration.rb +5 -5
data/lib/reek/configuration/configuration_converter.rb +1 -1
data/lib/reek/configuration/configuration_file_finder.rb +5 -4
data/lib/reek/configuration/default_directive.rb +1 -1
data/lib/reek/configuration/directory_directives.rb +1 -1
data/lib/reek/configuration/excluded_paths.rb +1 -1
data/lib/reek/configuration/schema.rb +177 -0
data/lib/reek/configuration/schema_validator.rb +12 -13
data/lib/reek/context/attribute_context.rb +1 -1
data/lib/reek/context/method_context.rb +1 -1
data/lib/reek/context/module_context.rb +4 -0
data/lib/reek/context/send_context.rb +7 -1
data/lib/reek/documentation_link.rb +3 -5
data/lib/reek/errors/bad_detector_configuration_key_in_comment_error.rb +2 -2
data/lib/reek/errors/bad_detector_in_comment_error.rb +2 -2
data/lib/reek/errors/encoding_error.rb +1 -1
data/lib/reek/errors/garbage_detector_configuration_in_comment_error.rb +2 -2
data/lib/reek/errors/incomprehensible_source_error.rb +1 -1
data/lib/reek/errors/legacy_comment_separator_error.rb +2 -2
data/lib/reek/errors/syntax_error.rb +1 -1
data/lib/reek/rake/task.rb +5 -5
data/lib/reek/smell_detectors/class_variable.rb +2 -2
data/lib/reek/smell_detectors/control_parameter_helpers/candidate.rb +6 -6
data/lib/reek/smell_detectors/control_parameter_helpers/control_parameter_finder.rb +1 -1
data/lib/reek/smell_detectors/duplicate_method_call.rb +5 -5
data/lib/reek/smell_detectors/instance_variable_assumption.rb +8 -8
data/lib/reek/smell_detectors/nested_iterators.rb +4 -3
data/lib/reek/smell_detectors/unused_private_method.rb +3 -2
data/lib/reek/spec/should_reek_of.rb +7 -7
data/lib/reek/spec.rb +1 -1
data/lib/reek/version.rb +1 -1
data/reek.gemspec +4 -3
metadata +30 -16
data/lib/reek/configuration/schema.yml +0 -210
/data/lib/reek/{report/code_climate.rb → code_climate.rb} +0 -0

data/lib/reek/configuration/schema.yml DELETED Viewed

@@ -1,210 +0,0 @@
----
-type: map
-mapping:
-  "detectors":
-    type: map
-    mapping: &all_detectors
-      Attribute:
-        type: map
-        mapping: &detector_base
-          "enabled":
-            type: bool
-          "exclude":
-            type: seq
-            sequence:
-              - type: str
-      BooleanParameter:
-        type: map
-        mapping:
-          <<: *detector_base
-      ClassVariable:
-        type: map
-        mapping:
-          <<: *detector_base
-      ControlParameter:
-        type: map
-        mapping:
-          <<: *detector_base
-      DataClump:
-        type: map
-        mapping:
-          <<: *detector_base
-          max_copies:
-            type: number
-          min_clump_size:
-            type: number
-      DuplicateMethodCall:
-        type: map
-        mapping:
-          <<: *detector_base
-          max_calls:
-            type: number
-          allow_calls:
-            type: seq
-            sequence:
-              - type: str
-      FeatureEnvy:
-        type: map
-        mapping:
-          <<: *detector_base
-      InstanceVariableAssumption:
-        type: map
-        mapping:
-          <<: *detector_base
-      IrresponsibleModule:
-        type: map
-        mapping:
-          <<: *detector_base
-      LongParameterList:
-        type: map
-        mapping:
-          <<: *detector_base
-          max_params:
-            type: number
-          overrides:
-            type: map
-            mapping:
-              initialize:
-                type: map
-                mapping:
-                  max_params:
-                    type: number
-      LongYieldList:
-        type: map
-        mapping:
-          <<: *detector_base
-          max_params:
-            type: number
-      ManualDispatch:
-        type: map
-        mapping:
-          <<: *detector_base
-      MissingSafeMethod:
-        type: map
-        mapping:
-          <<: *detector_base
-      ModuleInitialize:
-        type: map
-        mapping:
-          <<: *detector_base
-      NestedIterators:
-        type: map
-        mapping:
-          <<: *detector_base
-          max_allowed_nesting:
-            type: number
-          ignore_iterators:
-            type: seq
-            sequence:
-              - type: str
-      NilCheck:
-        type: map
-        mapping:
-          <<: *detector_base
-      RepeatedConditional:
-        type: map
-        mapping:
-          <<: *detector_base
-          max_ifs:
-            type: number
-      SubclassedFromCoreClass:
-        type: map
-        mapping:
-          <<: *detector_base
-      Syntax:
-        type: map
-        mapping:
-          <<: *detector_base
-      TooManyConstants:
-        type: map
-        mapping:
-          <<: *detector_base
-          max_constants:
-            type: number
-      TooManyInstanceVariables:
-        type: map
-        mapping:
-          <<: *detector_base
-          max_instance_variables:
-            type: number
-      TooManyMethods:
-        type: map
-        mapping:
-          <<: *detector_base
-          max_methods:
-            type: number
-      TooManyStatements:
-        type: map
-        mapping:
-          <<: *detector_base
-          max_statements:
-            type: number
-      UncommunicativeMethodName:
-        type: map
-        mapping:
-          <<: *detector_base
-          reject: &reject_settings
-            type: seq
-            sequence:
-              - type: str
-          accept: &accept_settings
-            type: seq
-            sequence:
-              - type: str
-      UncommunicativeModuleName:
-        type: map
-        mapping:
-          <<: *detector_base
-          reject: *reject_settings
-          accept: *accept_settings
-      UncommunicativeParameterName:
-        type: map
-        mapping:
-          <<: *detector_base
-          reject: *reject_settings
-          accept: *accept_settings
-      UncommunicativeVariableName:
-        type: map
-        mapping:
-          <<: *detector_base
-          reject: *reject_settings
-          accept: *accept_settings
-      UnusedParameters:
-        type: map
-        mapping:
-          <<: *detector_base
-      UnusedPrivateMethod:
-        type: map
-        mapping:
-          <<: *detector_base
-      UtilityFunction:
-        type: map
-        mapping:
-          <<: *detector_base
-          public_methods_only:
-            type: bool
-  "directories":
-    type: map
-    mapping:
-      # For any given key that is not matched somewhere else we'll apply the schema below.
-      # So this will just slurp in every directory we throw at it and then validate everything under
-      # it against all detectors - for instance:
-      #
-      #  directories:
-      #    "web_app/app/controllers":
-      #      NestedIterators:
-      #        enabled: false
-      #    "web_app/app/helpers":
-      #      UtilityFunction:
-      #        enabled: false
-      #
-      # For details check out: http://www.kuwata-lab.com/kwalify/ruby/users-guide.02.html#tips-default
-      =:
-        type: map
-        mapping: *all_detectors
-  "exclude_paths":
-    type: seq
-    sequence:
-      - type: str

/data/lib/reek/{report/code_climate.rb → code_climate.rb} RENAMED Viewed

File without changes