reek 6.0.2 → 6.5.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 (280) hide show
  1. checksums.yaml +4 -4
  2. data/.github/dependabot.yml +13 -0
  3. data/.github/workflows/ruby.yml +58 -0
  4. data/.gitignore +0 -1
  5. data/.rubocop.yml +7 -20
  6. data/.rubocop_todo.yml +6 -4
  7. data/.simplecov +2 -0
  8. data/CHANGELOG.md +114 -0
  9. data/CONTRIBUTING.md +10 -10
  10. data/Dockerfile +1 -1
  11. data/Gemfile +17 -24
  12. data/README.md +38 -40
  13. data/Rakefile +2 -0
  14. data/bin/code_climate_reek +56 -8
  15. data/docs/yard_plugin.rb +3 -1
  16. data/lib/reek/ast/ast_node_class_map.rb +1 -1
  17. data/lib/reek/ast/node.rb +16 -10
  18. data/lib/reek/ast/sexp_extensions/arguments.rb +20 -0
  19. data/lib/reek/ast/sexp_extensions/case.rb +1 -1
  20. data/lib/reek/ast/sexp_extensions/if.rb +1 -1
  21. data/lib/reek/ast/sexp_extensions/send.rb +22 -7
  22. data/lib/reek/cli/command/todo_list_command.rb +3 -3
  23. data/lib/reek/cli/options.rb +8 -8
  24. data/lib/reek/{report/code_climate → code_climate}/code_climate_configuration.rb +1 -1
  25. data/lib/reek/{report/code_climate → code_climate}/code_climate_configuration.yml +41 -41
  26. data/lib/reek/{report/code_climate → code_climate}/code_climate_fingerprint.rb +2 -2
  27. data/lib/reek/{report/code_climate → code_climate}/code_climate_formatter.rb +2 -4
  28. data/lib/reek/{report/code_climate → code_climate}/code_climate_report.rb +3 -3
  29. data/lib/reek/code_comment.rb +25 -20
  30. data/lib/reek/configuration/app_configuration.rb +5 -5
  31. data/lib/reek/configuration/configuration_converter.rb +1 -1
  32. data/lib/reek/configuration/configuration_file_finder.rb +5 -4
  33. data/lib/reek/configuration/default_directive.rb +1 -1
  34. data/lib/reek/configuration/directory_directives.rb +1 -1
  35. data/lib/reek/configuration/excluded_paths.rb +3 -2
  36. data/lib/reek/configuration/schema.rb +177 -0
  37. data/lib/reek/configuration/schema_validator.rb +12 -13
  38. data/lib/reek/context/attribute_context.rb +1 -1
  39. data/lib/reek/context/code_context.rb +3 -3
  40. data/lib/reek/context/method_context.rb +1 -1
  41. data/lib/reek/context/module_context.rb +4 -0
  42. data/lib/reek/context/refinement_context.rb +16 -0
  43. data/lib/reek/context/send_context.rb +7 -1
  44. data/lib/reek/context_builder.rb +17 -3
  45. data/lib/reek/documentation_link.rb +3 -5
  46. data/lib/reek/errors/bad_detector_configuration_key_in_comment_error.rb +2 -2
  47. data/lib/reek/errors/bad_detector_in_comment_error.rb +2 -2
  48. data/lib/reek/errors/encoding_error.rb +1 -1
  49. data/lib/reek/errors/garbage_detector_configuration_in_comment_error.rb +2 -2
  50. data/lib/reek/errors/incomprehensible_source_error.rb +1 -1
  51. data/lib/reek/errors/legacy_comment_separator_error.rb +2 -2
  52. data/lib/reek/errors/syntax_error.rb +1 -1
  53. data/lib/reek/rake/task.rb +5 -5
  54. data/lib/reek/report/github_report.rb +55 -0
  55. data/lib/reek/report/text_report.rb +1 -1
  56. data/lib/reek/report.rb +7 -5
  57. data/lib/reek/smell_detectors/base_detector.rb +1 -1
  58. data/lib/reek/smell_detectors/class_variable.rb +2 -2
  59. data/lib/reek/smell_detectors/control_parameter_helpers/candidate.rb +6 -6
  60. data/lib/reek/smell_detectors/control_parameter_helpers/control_parameter_finder.rb +1 -1
  61. data/lib/reek/smell_detectors/data_clump.rb +1 -1
  62. data/lib/reek/smell_detectors/duplicate_method_call.rb +5 -5
  63. data/lib/reek/smell_detectors/instance_variable_assumption.rb +8 -8
  64. data/lib/reek/smell_detectors/nested_iterators.rb +4 -3
  65. data/lib/reek/smell_detectors/uncommunicative_module_name.rb +1 -1
  66. data/lib/reek/smell_detectors/unused_private_method.rb +3 -2
  67. data/lib/reek/smell_warning.rb +1 -1
  68. data/lib/reek/source/source_code.rb +11 -5
  69. data/lib/reek/source/source_locator.rb +1 -3
  70. data/lib/reek/spec/should_reek_of.rb +11 -9
  71. data/lib/reek/spec.rb +1 -1
  72. data/lib/reek/version.rb +2 -2
  73. data/reek.gemspec +33 -25
  74. metadata +41 -238
  75. data/.travis.yml +0 -40
  76. data/docs/API.md +0 -174
  77. data/docs/Attribute.md +0 -39
  78. data/docs/Basic-Smell-Options.md +0 -85
  79. data/docs/Boolean-Parameter.md +0 -54
  80. data/docs/Class-Variable.md +0 -40
  81. data/docs/Code-Smells.md +0 -39
  82. data/docs/Command-Line-Options.md +0 -119
  83. data/docs/Control-Couple.md +0 -26
  84. data/docs/Control-Parameter.md +0 -32
  85. data/docs/Data-Clump.md +0 -46
  86. data/docs/Duplicate-Method-Call.md +0 -264
  87. data/docs/Feature-Envy.md +0 -93
  88. data/docs/How-To-Write-New-Detectors.md +0 -132
  89. data/docs/How-reek-works-internally.md +0 -114
  90. data/docs/Instance-Variable-Assumption.md +0 -163
  91. data/docs/Irresponsible-Module.md +0 -47
  92. data/docs/Large-Class.md +0 -16
  93. data/docs/Long-Parameter-List.md +0 -39
  94. data/docs/Long-Yield-List.md +0 -37
  95. data/docs/Manual-Dispatch.md +0 -30
  96. data/docs/Missing-Safe-Method.md +0 -92
  97. data/docs/Module-Initialize.md +0 -62
  98. data/docs/Nested-Iterators.md +0 -59
  99. data/docs/Nil-Check.md +0 -47
  100. data/docs/RSpec-matchers.md +0 -129
  101. data/docs/Rake-Task.md +0 -66
  102. data/docs/Reek-4-to-Reek-5-migration.md +0 -188
  103. data/docs/Reek-Driven-Development.md +0 -46
  104. data/docs/Repeated-Conditional.md +0 -47
  105. data/docs/Simulated-Polymorphism.md +0 -16
  106. data/docs/Smell-Suppression.md +0 -96
  107. data/docs/Style-Guide.md +0 -19
  108. data/docs/Subclassed-From-Core-Class.md +0 -79
  109. data/docs/Too-Many-Constants.md +0 -37
  110. data/docs/Too-Many-Instance-Variables.md +0 -43
  111. data/docs/Too-Many-Methods.md +0 -56
  112. data/docs/Too-Many-Statements.md +0 -54
  113. data/docs/Uncommunicative-Method-Name.md +0 -94
  114. data/docs/Uncommunicative-Module-Name.md +0 -92
  115. data/docs/Uncommunicative-Name.md +0 -18
  116. data/docs/Uncommunicative-Parameter-Name.md +0 -90
  117. data/docs/Uncommunicative-Variable-Name.md +0 -96
  118. data/docs/Unused-Parameters.md +0 -28
  119. data/docs/Unused-Private-Method.md +0 -101
  120. data/docs/Utility-Function.md +0 -56
  121. data/docs/Versioning-Policy.md +0 -7
  122. data/docs/YAML-Reports.md +0 -93
  123. data/docs/defaults.reek.yml +0 -129
  124. data/docs/templates/default/docstring/html/public_api_marker.erb +0 -3
  125. data/docs/templates/default/docstring/setup.rb +0 -37
  126. data/docs/templates/default/fulldoc/html/css/common.css +0 -1
  127. data/features/command_line_interface/basic_usage.feature +0 -15
  128. data/features/command_line_interface/options.feature +0 -123
  129. data/features/command_line_interface/show_progress.feature +0 -33
  130. data/features/command_line_interface/smell_selection.feature +0 -15
  131. data/features/command_line_interface/smells_count.feature +0 -38
  132. data/features/command_line_interface/stdin.feature +0 -65
  133. data/features/configuration_files/accept_setting.feature +0 -87
  134. data/features/configuration_files/directory_specific_directives.feature +0 -274
  135. data/features/configuration_files/exclude_directives.feature +0 -35
  136. data/features/configuration_files/exclude_paths_directives.feature +0 -42
  137. data/features/configuration_files/masking_smells.feature +0 -94
  138. data/features/configuration_files/mix_accept_reject_setting.feature +0 -84
  139. data/features/configuration_files/reject_setting.feature +0 -89
  140. data/features/configuration_files/schema_validation.feature +0 -59
  141. data/features/configuration_files/show_configuration_file.feature +0 -44
  142. data/features/configuration_files/unused_private_method.feature +0 -68
  143. data/features/configuration_loading.feature +0 -91
  144. data/features/configuration_via_source_comments/erroneous_source_comments.feature +0 -68
  145. data/features/configuration_via_source_comments/well_formed_source_comments.feature +0 -116
  146. data/features/locales.feature +0 -32
  147. data/features/programmatic_access.feature +0 -41
  148. data/features/rake_task/rake_task.feature +0 -138
  149. data/features/reports/codeclimate.feature +0 -59
  150. data/features/reports/json.feature +0 -59
  151. data/features/reports/reports.feature +0 -219
  152. data/features/reports/yaml.feature +0 -52
  153. data/features/rspec_matcher.feature +0 -41
  154. data/features/samples.feature +0 -305
  155. data/features/step_definitions/.rubocop.yml +0 -5
  156. data/features/step_definitions/reek_steps.rb +0 -102
  157. data/features/step_definitions/sample_file_steps.rb +0 -63
  158. data/features/support/env.rb +0 -33
  159. data/features/todo_list.feature +0 -108
  160. data/lib/reek/configuration/schema.yml +0 -210
  161. data/samples/checkstyle.xml +0 -7
  162. data/samples/clean_source/clean.rb +0 -6
  163. data/samples/configuration/accepts_rejects_and_excludes_for_detectors.reek.yml +0 -29
  164. data/samples/configuration/accepts_rejects_and_excludes_for_directory_directives.reek.yml +0 -30
  165. data/samples/configuration/corrupt.reek +0 -1
  166. data/samples/configuration/empty.reek +0 -0
  167. data/samples/configuration/full_configuration.reek +0 -13
  168. data/samples/configuration/full_mask.reek +0 -6
  169. data/samples/configuration/home/home.reek.yml +0 -4
  170. data/samples/configuration/partial_mask.reek +0 -4
  171. data/samples/configuration/regular_configuration/.reek.yml +0 -4
  172. data/samples/configuration/regular_configuration/empty_sub_directory/.gitignore +0 -0
  173. data/samples/configuration/with_excluded_paths.reek +0 -5
  174. data/samples/no_config_file/.keep +0 -0
  175. data/samples/paths.rb +0 -5
  176. data/samples/smelly_source/inline.rb +0 -704
  177. data/samples/smelly_source/optparse.rb +0 -1788
  178. data/samples/smelly_source/redcloth.rb +0 -1130
  179. data/samples/smelly_source/ruby.rb +0 -368
  180. data/samples/smelly_source/smelly.rb +0 -7
  181. data/samples/source_with_exclude_paths/ignore_me/uncommunicative_method_name.rb +0 -5
  182. data/samples/source_with_exclude_paths/nested/ignore_me_as_well/irresponsible_module.rb +0 -2
  183. data/samples/source_with_exclude_paths/nested/uncommunicative_parameter_name.rb +0 -6
  184. data/samples/source_with_exclude_paths/nested/uncommunicative_variable_name.rb +0 -6
  185. data/samples/source_with_hidden_directories/.hidden/hidden.rb +0 -1
  186. data/samples/source_with_hidden_directories/not_hidden.rb +0 -1
  187. data/samples/source_with_non_ruby_files/gibberish +0 -1
  188. data/samples/source_with_non_ruby_files/python_source.py +0 -1
  189. data/samples/source_with_non_ruby_files/ruby.rb +0 -6
  190. data/spec/performance/reek/smell_detectors/runtime_speed_spec.rb +0 -15
  191. data/spec/quality/documentation_spec.rb +0 -41
  192. data/spec/quality/reek_source_spec.rb +0 -11
  193. data/spec/reek/ast/node_spec.rb +0 -211
  194. data/spec/reek/ast/object_refs_spec.rb +0 -83
  195. data/spec/reek/ast/reference_collector_spec.rb +0 -47
  196. data/spec/reek/ast/sexp_extensions_spec.rb +0 -498
  197. data/spec/reek/cli/application_spec.rb +0 -168
  198. data/spec/reek/cli/command/report_command_spec.rb +0 -44
  199. data/spec/reek/cli/command/todo_list_command_spec.rb +0 -86
  200. data/spec/reek/cli/options_spec.rb +0 -51
  201. data/spec/reek/cli/silencer_spec.rb +0 -28
  202. data/spec/reek/code_comment_spec.rb +0 -184
  203. data/spec/reek/configuration/app_configuration_spec.rb +0 -195
  204. data/spec/reek/configuration/configuration_file_finder_spec.rb +0 -230
  205. data/spec/reek/configuration/default_directive_spec.rb +0 -13
  206. data/spec/reek/configuration/directory_directives_spec.rb +0 -122
  207. data/spec/reek/configuration/excluded_paths_spec.rb +0 -16
  208. data/spec/reek/configuration/rake_task_converter_spec.rb +0 -33
  209. data/spec/reek/configuration/schema_validator_spec.rb +0 -165
  210. data/spec/reek/context/code_context_spec.rb +0 -192
  211. data/spec/reek/context/ghost_context_spec.rb +0 -60
  212. data/spec/reek/context/method_context_spec.rb +0 -72
  213. data/spec/reek/context/module_context_spec.rb +0 -55
  214. data/spec/reek/context/root_context_spec.rb +0 -12
  215. data/spec/reek/context/statement_counter_spec.rb +0 -24
  216. data/spec/reek/context_builder_spec.rb +0 -457
  217. data/spec/reek/detector_repository_spec.rb +0 -22
  218. data/spec/reek/documentation_link_spec.rb +0 -20
  219. data/spec/reek/errors/base_error_spec.rb +0 -13
  220. data/spec/reek/examiner_spec.rb +0 -309
  221. data/spec/reek/logging_error_handler_spec.rb +0 -24
  222. data/spec/reek/rake/task_spec.rb +0 -56
  223. data/spec/reek/report/code_climate/code_climate_configuration_spec.rb +0 -22
  224. data/spec/reek/report/code_climate/code_climate_fingerprint_spec.rb +0 -126
  225. data/spec/reek/report/code_climate/code_climate_formatter_spec.rb +0 -51
  226. data/spec/reek/report/code_climate/code_climate_report_spec.rb +0 -56
  227. data/spec/reek/report/html_report_spec.rb +0 -19
  228. data/spec/reek/report/json_report_spec.rb +0 -58
  229. data/spec/reek/report/location_formatter_spec.rb +0 -32
  230. data/spec/reek/report/progress_formatter_spec.rb +0 -68
  231. data/spec/reek/report/text_report_spec.rb +0 -89
  232. data/spec/reek/report/xml_report_spec.rb +0 -24
  233. data/spec/reek/report/yaml_report_spec.rb +0 -55
  234. data/spec/reek/report_spec.rb +0 -28
  235. data/spec/reek/smell_configuration_spec.rb +0 -56
  236. data/spec/reek/smell_detectors/attribute_spec.rb +0 -197
  237. data/spec/reek/smell_detectors/base_detector_spec.rb +0 -50
  238. data/spec/reek/smell_detectors/boolean_parameter_spec.rb +0 -93
  239. data/spec/reek/smell_detectors/class_variable_spec.rb +0 -106
  240. data/spec/reek/smell_detectors/control_parameter_spec.rb +0 -300
  241. data/spec/reek/smell_detectors/data_clump_spec.rb +0 -134
  242. data/spec/reek/smell_detectors/duplicate_method_call_spec.rb +0 -211
  243. data/spec/reek/smell_detectors/feature_envy_spec.rb +0 -295
  244. data/spec/reek/smell_detectors/instance_variable_assumption_spec.rb +0 -96
  245. data/spec/reek/smell_detectors/irresponsible_module_spec.rb +0 -226
  246. data/spec/reek/smell_detectors/long_parameter_list_spec.rb +0 -61
  247. data/spec/reek/smell_detectors/long_yield_list_spec.rb +0 -49
  248. data/spec/reek/smell_detectors/manual_dispatch_spec.rb +0 -75
  249. data/spec/reek/smell_detectors/missing_safe_method_spec.rb +0 -68
  250. data/spec/reek/smell_detectors/module_initialize_spec.rb +0 -77
  251. data/spec/reek/smell_detectors/nested_iterators_spec.rb +0 -333
  252. data/spec/reek/smell_detectors/nil_check_spec.rb +0 -100
  253. data/spec/reek/smell_detectors/repeated_conditional_spec.rb +0 -100
  254. data/spec/reek/smell_detectors/subclassed_from_core_class_spec.rb +0 -77
  255. data/spec/reek/smell_detectors/too_many_constants_spec.rb +0 -144
  256. data/spec/reek/smell_detectors/too_many_instance_variables_spec.rb +0 -132
  257. data/spec/reek/smell_detectors/too_many_methods_spec.rb +0 -54
  258. data/spec/reek/smell_detectors/too_many_statements_spec.rb +0 -90
  259. data/spec/reek/smell_detectors/uncommunicative_method_name_spec.rb +0 -78
  260. data/spec/reek/smell_detectors/uncommunicative_module_name_spec.rb +0 -78
  261. data/spec/reek/smell_detectors/uncommunicative_parameter_name_spec.rb +0 -147
  262. data/spec/reek/smell_detectors/uncommunicative_variable_name_spec.rb +0 -201
  263. data/spec/reek/smell_detectors/unused_parameters_spec.rb +0 -114
  264. data/spec/reek/smell_detectors/unused_private_method_spec.rb +0 -205
  265. data/spec/reek/smell_detectors/utility_function_spec.rb +0 -293
  266. data/spec/reek/smell_warning_spec.rb +0 -137
  267. data/spec/reek/source/source_code_spec.rb +0 -79
  268. data/spec/reek/source/source_locator_spec.rb +0 -166
  269. data/spec/reek/spec/should_reek_of_spec.rb +0 -153
  270. data/spec/reek/spec/should_reek_only_of_spec.rb +0 -91
  271. data/spec/reek/spec/should_reek_spec.rb +0 -52
  272. data/spec/reek/spec/smell_matcher_spec.rb +0 -87
  273. data/spec/reek/tree_dresser_spec.rb +0 -46
  274. data/spec/spec_helper.rb +0 -110
  275. data/tasks/configuration.rake +0 -18
  276. data/tasks/console.rake +0 -5
  277. data/tasks/reek.rake +0 -6
  278. data/tasks/rubocop.rake +0 -11
  279. data/tasks/test.rake +0 -32
  280. /data/lib/reek/{report/code_climate.rb → code_climate.rb} +0 -0
metadata CHANGED
@@ -1,7 +1,7 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: reek
3
3
  version: !ruby/object:Gem::Version
4
- version: 6.0.2
4
+ version: 6.5.0
5
5
  platform: ruby
6
6
  authors:
7
7
  - Kevin Rutherford
@@ -11,62 +11,50 @@ authors:
11
11
  autorequire:
12
12
  bindir: bin
13
13
  cert_chain: []
14
- date: 2020-10-17 00:00:00.000000000 Z
14
+ date: 2025-03-24 00:00:00.000000000 Z
15
15
  dependencies:
16
16
  - !ruby/object:Gem::Dependency
17
- name: kwalify
17
+ name: dry-schema
18
18
  requirement: !ruby/object:Gem::Requirement
19
19
  requirements:
20
20
  - - "~>"
21
21
  - !ruby/object:Gem::Version
22
- version: 0.7.0
22
+ version: '1.13'
23
23
  type: :runtime
24
24
  prerelease: false
25
25
  version_requirements: !ruby/object:Gem::Requirement
26
26
  requirements:
27
27
  - - "~>"
28
28
  - !ruby/object:Gem::Version
29
- version: 0.7.0
29
+ version: '1.13'
30
30
  - !ruby/object:Gem::Dependency
31
- name: parser
31
+ name: logger
32
32
  requirement: !ruby/object:Gem::Requirement
33
33
  requirements:
34
- - - "<"
35
- - !ruby/object:Gem::Version
36
- version: '2.8'
37
- - - ">="
38
- - !ruby/object:Gem::Version
39
- version: 2.5.0.0
40
- - - "!="
34
+ - - "~>"
41
35
  - !ruby/object:Gem::Version
42
- version: 2.5.1.1
36
+ version: '1.6'
43
37
  type: :runtime
44
38
  prerelease: false
45
39
  version_requirements: !ruby/object:Gem::Requirement
46
40
  requirements:
47
- - - "<"
48
- - !ruby/object:Gem::Version
49
- version: '2.8'
50
- - - ">="
51
- - !ruby/object:Gem::Version
52
- version: 2.5.0.0
53
- - - "!="
41
+ - - "~>"
54
42
  - !ruby/object:Gem::Version
55
- version: 2.5.1.1
43
+ version: '1.6'
56
44
  - !ruby/object:Gem::Dependency
57
- name: psych
45
+ name: parser
58
46
  requirement: !ruby/object:Gem::Requirement
59
47
  requirements:
60
48
  - - "~>"
61
49
  - !ruby/object:Gem::Version
62
- version: '3.1'
50
+ version: 3.3.0
63
51
  type: :runtime
64
52
  prerelease: false
65
53
  version_requirements: !ruby/object:Gem::Requirement
66
54
  requirements:
67
55
  - - "~>"
68
56
  - !ruby/object:Gem::Version
69
- version: '3.1'
57
+ version: 3.3.0
70
58
  - !ruby/object:Gem::Dependency
71
59
  name: rainbow
72
60
  requirement: !ruby/object:Gem::Requirement
@@ -87,6 +75,20 @@ dependencies:
87
75
  - - "<"
88
76
  - !ruby/object:Gem::Version
89
77
  version: '4.0'
78
+ - !ruby/object:Gem::Dependency
79
+ name: rexml
80
+ requirement: !ruby/object:Gem::Requirement
81
+ requirements:
82
+ - - "~>"
83
+ - !ruby/object:Gem::Version
84
+ version: '3.1'
85
+ type: :runtime
86
+ prerelease: false
87
+ version_requirements: !ruby/object:Gem::Requirement
88
+ requirements:
89
+ - - "~>"
90
+ - !ruby/object:Gem::Version
91
+ version: '3.1'
90
92
  description: Reek is a tool that examines Ruby classes, modules and methods and reports
91
93
  any code smells it finds.
92
94
  email:
@@ -100,12 +102,13 @@ extra_rdoc_files:
100
102
  - License.txt
101
103
  files:
102
104
  - ".dockerignore"
105
+ - ".github/dependabot.yml"
106
+ - ".github/workflows/ruby.yml"
103
107
  - ".gitignore"
104
108
  - ".reek.yml"
105
109
  - ".rubocop.yml"
106
110
  - ".rubocop_todo.yml"
107
111
  - ".simplecov"
108
- - ".travis.yml"
109
112
  - ".yardopts"
110
113
  - CHANGELOG.md
111
114
  - CONTRIBUTING.md
@@ -116,92 +119,8 @@ files:
116
119
  - Rakefile
117
120
  - bin/code_climate_reek
118
121
  - bin/reek
119
- - docs/API.md
120
- - docs/Attribute.md
121
- - docs/Basic-Smell-Options.md
122
- - docs/Boolean-Parameter.md
123
- - docs/Class-Variable.md
124
- - docs/Code-Smells.md
125
- - docs/Command-Line-Options.md
126
- - docs/Control-Couple.md
127
- - docs/Control-Parameter.md
128
- - docs/Data-Clump.md
129
- - docs/Duplicate-Method-Call.md
130
- - docs/Feature-Envy.md
131
- - docs/How-To-Write-New-Detectors.md
132
- - docs/How-reek-works-internally.md
133
- - docs/Instance-Variable-Assumption.md
134
- - docs/Irresponsible-Module.md
135
- - docs/Large-Class.md
136
- - docs/Long-Parameter-List.md
137
- - docs/Long-Yield-List.md
138
- - docs/Manual-Dispatch.md
139
- - docs/Missing-Safe-Method.md
140
- - docs/Module-Initialize.md
141
- - docs/Nested-Iterators.md
142
- - docs/Nil-Check.md
143
- - docs/RSpec-matchers.md
144
- - docs/Rake-Task.md
145
- - docs/Reek-4-to-Reek-5-migration.md
146
- - docs/Reek-Driven-Development.md
147
- - docs/Repeated-Conditional.md
148
- - docs/Simulated-Polymorphism.md
149
- - docs/Smell-Suppression.md
150
- - docs/Style-Guide.md
151
- - docs/Subclassed-From-Core-Class.md
152
- - docs/Too-Many-Constants.md
153
- - docs/Too-Many-Instance-Variables.md
154
- - docs/Too-Many-Methods.md
155
- - docs/Too-Many-Statements.md
156
- - docs/Uncommunicative-Method-Name.md
157
- - docs/Uncommunicative-Module-Name.md
158
- - docs/Uncommunicative-Name.md
159
- - docs/Uncommunicative-Parameter-Name.md
160
- - docs/Uncommunicative-Variable-Name.md
161
- - docs/Unused-Parameters.md
162
- - docs/Unused-Private-Method.md
163
- - docs/Utility-Function.md
164
- - docs/Versioning-Policy.md
165
- - docs/YAML-Reports.md
166
- - docs/defaults.reek.yml
167
- - docs/templates/default/docstring/html/public_api_marker.erb
168
- - docs/templates/default/docstring/setup.rb
169
- - docs/templates/default/fulldoc/html/css/common.css
170
122
  - docs/yard_plugin.rb
171
123
  - engine.json
172
- - features/command_line_interface/basic_usage.feature
173
- - features/command_line_interface/options.feature
174
- - features/command_line_interface/show_progress.feature
175
- - features/command_line_interface/smell_selection.feature
176
- - features/command_line_interface/smells_count.feature
177
- - features/command_line_interface/stdin.feature
178
- - features/configuration_files/accept_setting.feature
179
- - features/configuration_files/directory_specific_directives.feature
180
- - features/configuration_files/exclude_directives.feature
181
- - features/configuration_files/exclude_paths_directives.feature
182
- - features/configuration_files/masking_smells.feature
183
- - features/configuration_files/mix_accept_reject_setting.feature
184
- - features/configuration_files/reject_setting.feature
185
- - features/configuration_files/schema_validation.feature
186
- - features/configuration_files/show_configuration_file.feature
187
- - features/configuration_files/unused_private_method.feature
188
- - features/configuration_loading.feature
189
- - features/configuration_via_source_comments/erroneous_source_comments.feature
190
- - features/configuration_via_source_comments/well_formed_source_comments.feature
191
- - features/locales.feature
192
- - features/programmatic_access.feature
193
- - features/rake_task/rake_task.feature
194
- - features/reports/codeclimate.feature
195
- - features/reports/json.feature
196
- - features/reports/reports.feature
197
- - features/reports/yaml.feature
198
- - features/rspec_matcher.feature
199
- - features/samples.feature
200
- - features/step_definitions/.rubocop.yml
201
- - features/step_definitions/reek_steps.rb
202
- - features/step_definitions/sample_file_steps.rb
203
- - features/support/env.rb
204
- - features/todo_list.feature
205
124
  - lib/reek.rb
206
125
  - lib/reek/ast/ast_node_class_map.rb
207
126
  - lib/reek/ast/builder.rb
@@ -234,6 +153,12 @@ files:
234
153
  - lib/reek/cli/options.rb
235
154
  - lib/reek/cli/silencer.rb
236
155
  - lib/reek/cli/status.rb
156
+ - lib/reek/code_climate.rb
157
+ - lib/reek/code_climate/code_climate_configuration.rb
158
+ - lib/reek/code_climate/code_climate_configuration.yml
159
+ - lib/reek/code_climate/code_climate_fingerprint.rb
160
+ - lib/reek/code_climate/code_climate_formatter.rb
161
+ - lib/reek/code_climate/code_climate_report.rb
237
162
  - lib/reek/code_comment.rb
238
163
  - lib/reek/configuration/app_configuration.rb
239
164
  - lib/reek/configuration/configuration_converter.rb
@@ -243,7 +168,7 @@ files:
243
168
  - lib/reek/configuration/directory_directives.rb
244
169
  - lib/reek/configuration/excluded_paths.rb
245
170
  - lib/reek/configuration/rake_task_converter.rb
246
- - lib/reek/configuration/schema.yml
171
+ - lib/reek/configuration/schema.rb
247
172
  - lib/reek/configuration/schema_validator.rb
248
173
  - lib/reek/context/attribute_context.rb
249
174
  - lib/reek/context/class_context.rb
@@ -251,6 +176,7 @@ files:
251
176
  - lib/reek/context/ghost_context.rb
252
177
  - lib/reek/context/method_context.rb
253
178
  - lib/reek/context/module_context.rb
179
+ - lib/reek/context/refinement_context.rb
254
180
  - lib/reek/context/root_context.rb
255
181
  - lib/reek/context/send_context.rb
256
182
  - lib/reek/context/singleton_attribute_context.rb
@@ -274,13 +200,8 @@ files:
274
200
  - lib/reek/rake/task.rb
275
201
  - lib/reek/report.rb
276
202
  - lib/reek/report/base_report.rb
277
- - lib/reek/report/code_climate.rb
278
- - lib/reek/report/code_climate/code_climate_configuration.rb
279
- - lib/reek/report/code_climate/code_climate_configuration.yml
280
- - lib/reek/report/code_climate/code_climate_fingerprint.rb
281
- - lib/reek/report/code_climate/code_climate_formatter.rb
282
- - lib/reek/report/code_climate/code_climate_report.rb
283
203
  - lib/reek/report/documentation_link_warning_formatter.rb
204
+ - lib/reek/report/github_report.rb
284
205
  - lib/reek/report/heading_formatter.rb
285
206
  - lib/reek/report/html_report.rb
286
207
  - lib/reek/report/html_report/html_report.html.erb
@@ -343,125 +264,6 @@ files:
343
264
  - logo/reek.text.png
344
265
  - logo/reek.text.svg
345
266
  - reek.gemspec
346
- - samples/checkstyle.xml
347
- - samples/clean_source/clean.rb
348
- - samples/configuration/accepts_rejects_and_excludes_for_detectors.reek.yml
349
- - samples/configuration/accepts_rejects_and_excludes_for_directory_directives.reek.yml
350
- - samples/configuration/corrupt.reek
351
- - samples/configuration/empty.reek
352
- - samples/configuration/full_configuration.reek
353
- - samples/configuration/full_mask.reek
354
- - samples/configuration/home/home.reek.yml
355
- - samples/configuration/partial_mask.reek
356
- - samples/configuration/regular_configuration/.reek.yml
357
- - samples/configuration/regular_configuration/empty_sub_directory/.gitignore
358
- - samples/configuration/with_excluded_paths.reek
359
- - samples/no_config_file/.keep
360
- - samples/paths.rb
361
- - samples/smelly_source/inline.rb
362
- - samples/smelly_source/optparse.rb
363
- - samples/smelly_source/redcloth.rb
364
- - samples/smelly_source/ruby.rb
365
- - samples/smelly_source/smelly.rb
366
- - samples/source_with_exclude_paths/ignore_me/uncommunicative_method_name.rb
367
- - samples/source_with_exclude_paths/nested/ignore_me_as_well/irresponsible_module.rb
368
- - samples/source_with_exclude_paths/nested/uncommunicative_parameter_name.rb
369
- - samples/source_with_exclude_paths/nested/uncommunicative_variable_name.rb
370
- - samples/source_with_hidden_directories/.hidden/hidden.rb
371
- - samples/source_with_hidden_directories/not_hidden.rb
372
- - samples/source_with_non_ruby_files/gibberish
373
- - samples/source_with_non_ruby_files/python_source.py
374
- - samples/source_with_non_ruby_files/ruby.rb
375
- - spec/performance/reek/smell_detectors/runtime_speed_spec.rb
376
- - spec/quality/documentation_spec.rb
377
- - spec/quality/reek_source_spec.rb
378
- - spec/reek/ast/node_spec.rb
379
- - spec/reek/ast/object_refs_spec.rb
380
- - spec/reek/ast/reference_collector_spec.rb
381
- - spec/reek/ast/sexp_extensions_spec.rb
382
- - spec/reek/cli/application_spec.rb
383
- - spec/reek/cli/command/report_command_spec.rb
384
- - spec/reek/cli/command/todo_list_command_spec.rb
385
- - spec/reek/cli/options_spec.rb
386
- - spec/reek/cli/silencer_spec.rb
387
- - spec/reek/code_comment_spec.rb
388
- - spec/reek/configuration/app_configuration_spec.rb
389
- - spec/reek/configuration/configuration_file_finder_spec.rb
390
- - spec/reek/configuration/default_directive_spec.rb
391
- - spec/reek/configuration/directory_directives_spec.rb
392
- - spec/reek/configuration/excluded_paths_spec.rb
393
- - spec/reek/configuration/rake_task_converter_spec.rb
394
- - spec/reek/configuration/schema_validator_spec.rb
395
- - spec/reek/context/code_context_spec.rb
396
- - spec/reek/context/ghost_context_spec.rb
397
- - spec/reek/context/method_context_spec.rb
398
- - spec/reek/context/module_context_spec.rb
399
- - spec/reek/context/root_context_spec.rb
400
- - spec/reek/context/statement_counter_spec.rb
401
- - spec/reek/context_builder_spec.rb
402
- - spec/reek/detector_repository_spec.rb
403
- - spec/reek/documentation_link_spec.rb
404
- - spec/reek/errors/base_error_spec.rb
405
- - spec/reek/examiner_spec.rb
406
- - spec/reek/logging_error_handler_spec.rb
407
- - spec/reek/rake/task_spec.rb
408
- - spec/reek/report/code_climate/code_climate_configuration_spec.rb
409
- - spec/reek/report/code_climate/code_climate_fingerprint_spec.rb
410
- - spec/reek/report/code_climate/code_climate_formatter_spec.rb
411
- - spec/reek/report/code_climate/code_climate_report_spec.rb
412
- - spec/reek/report/html_report_spec.rb
413
- - spec/reek/report/json_report_spec.rb
414
- - spec/reek/report/location_formatter_spec.rb
415
- - spec/reek/report/progress_formatter_spec.rb
416
- - spec/reek/report/text_report_spec.rb
417
- - spec/reek/report/xml_report_spec.rb
418
- - spec/reek/report/yaml_report_spec.rb
419
- - spec/reek/report_spec.rb
420
- - spec/reek/smell_configuration_spec.rb
421
- - spec/reek/smell_detectors/attribute_spec.rb
422
- - spec/reek/smell_detectors/base_detector_spec.rb
423
- - spec/reek/smell_detectors/boolean_parameter_spec.rb
424
- - spec/reek/smell_detectors/class_variable_spec.rb
425
- - spec/reek/smell_detectors/control_parameter_spec.rb
426
- - spec/reek/smell_detectors/data_clump_spec.rb
427
- - spec/reek/smell_detectors/duplicate_method_call_spec.rb
428
- - spec/reek/smell_detectors/feature_envy_spec.rb
429
- - spec/reek/smell_detectors/instance_variable_assumption_spec.rb
430
- - spec/reek/smell_detectors/irresponsible_module_spec.rb
431
- - spec/reek/smell_detectors/long_parameter_list_spec.rb
432
- - spec/reek/smell_detectors/long_yield_list_spec.rb
433
- - spec/reek/smell_detectors/manual_dispatch_spec.rb
434
- - spec/reek/smell_detectors/missing_safe_method_spec.rb
435
- - spec/reek/smell_detectors/module_initialize_spec.rb
436
- - spec/reek/smell_detectors/nested_iterators_spec.rb
437
- - spec/reek/smell_detectors/nil_check_spec.rb
438
- - spec/reek/smell_detectors/repeated_conditional_spec.rb
439
- - spec/reek/smell_detectors/subclassed_from_core_class_spec.rb
440
- - spec/reek/smell_detectors/too_many_constants_spec.rb
441
- - spec/reek/smell_detectors/too_many_instance_variables_spec.rb
442
- - spec/reek/smell_detectors/too_many_methods_spec.rb
443
- - spec/reek/smell_detectors/too_many_statements_spec.rb
444
- - spec/reek/smell_detectors/uncommunicative_method_name_spec.rb
445
- - spec/reek/smell_detectors/uncommunicative_module_name_spec.rb
446
- - spec/reek/smell_detectors/uncommunicative_parameter_name_spec.rb
447
- - spec/reek/smell_detectors/uncommunicative_variable_name_spec.rb
448
- - spec/reek/smell_detectors/unused_parameters_spec.rb
449
- - spec/reek/smell_detectors/unused_private_method_spec.rb
450
- - spec/reek/smell_detectors/utility_function_spec.rb
451
- - spec/reek/smell_warning_spec.rb
452
- - spec/reek/source/source_code_spec.rb
453
- - spec/reek/source/source_locator_spec.rb
454
- - spec/reek/spec/should_reek_of_spec.rb
455
- - spec/reek/spec/should_reek_only_of_spec.rb
456
- - spec/reek/spec/should_reek_spec.rb
457
- - spec/reek/spec/smell_matcher_spec.rb
458
- - spec/reek/tree_dresser_spec.rb
459
- - spec/spec_helper.rb
460
- - tasks/configuration.rake
461
- - tasks/console.rake
462
- - tasks/reek.rake
463
- - tasks/rubocop.rake
464
- - tasks/test.rake
465
267
  homepage: https://github.com/troessner/reek
466
268
  licenses:
467
269
  - MIT
@@ -471,6 +273,7 @@ metadata:
471
273
  bug_tracker_uri: https://github.com/troessner/reek/issues
472
274
  changelog_uri: https://github.com/troessner/reek/CHANGELOG.md
473
275
  documentation_uri: https://www.rubydoc.info/gems/reek
276
+ rubygems_mfa_required: 'true'
474
277
  post_install_message:
475
278
  rdoc_options:
476
279
  - "--main"
@@ -483,14 +286,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
483
286
  requirements:
484
287
  - - ">="
485
288
  - !ruby/object:Gem::Version
486
- version: 2.4.0
289
+ version: 3.1.0
487
290
  required_rubygems_version: !ruby/object:Gem::Requirement
488
291
  requirements:
489
292
  - - ">="
490
293
  - !ruby/object:Gem::Version
491
294
  version: '0'
492
295
  requirements: []
493
- rubygems_version: 3.1.2
296
+ rubygems_version: 3.0.3.1
494
297
  signing_key:
495
298
  specification_version: 4
496
299
  summary: Code smell detector for Ruby
data/.travis.yml DELETED
@@ -1,40 +0,0 @@
1
- language: ruby
2
-
3
- dist: xenial
4
-
5
- cache:
6
- bundler: true
7
-
8
- bundler_args: --without debugging
9
-
10
- script: bundle exec rake ci
11
-
12
- rvm:
13
- - 2.4
14
- - 2.5
15
- - 2.6
16
- - 2.7
17
- - jruby-9.2
18
- - jruby-head
19
- - ruby-head
20
-
21
- matrix:
22
- allow_failures:
23
- - rvm: jruby-head
24
- - rvm: ruby-head
25
- fast_finish: true
26
-
27
- notifications:
28
- email:
29
- - timo.roessner@googlemail.com
30
- - matijs@matijs.net
31
- - chastell@chastell.net
32
- irc: irc.freenode.org#reek
33
-
34
- branches:
35
- only:
36
- - master
37
-
38
- env:
39
- global:
40
- - JRUBY_OPTS="--dev"
data/docs/API.md DELETED
@@ -1,174 +0,0 @@
1
- # Using Reek inside your Ruby application
2
-
3
- ## Installation
4
-
5
- Either standalone via
6
-
7
- ```bash
8
- gem install reek
9
- ```
10
-
11
- or by adding
12
-
13
- ```
14
- gem 'reek'
15
- ```
16
-
17
- to your Gemfile.
18
-
19
- ## Quick start
20
-
21
- Code says more than a thousand words:
22
-
23
- ```ruby
24
- require 'reek'
25
-
26
- source = <<-RUBY
27
- class Dirty
28
- def m(a,b,c)
29
- puts a,b
30
- end
31
- end
32
- RUBY
33
-
34
- reporter = Reek::Report::TextReport.new
35
- examiner = Reek::Examiner.new source
36
- reporter.add_examiner examiner
37
- reporter.show
38
- ```
39
-
40
- This would output the following on STDOUT:
41
-
42
- ```
43
- string -- 5 warnings:
44
- Dirty has no descriptive comment (IrresponsibleModule)
45
- Dirty#m has the name 'm' (UncommunicativeMethodName)
46
- Dirty#m has the parameter name 'a' (UncommunicativeParameterName)
47
- Dirty#m has the parameter name 'b' (UncommunicativeParameterName)
48
- Dirty#m has unused parameter 'c' (UnusedParameters)
49
- ```
50
-
51
- Note that `Reek::Examiner.new` can take `source` as `String`, `Pathname`, `File` or `IO`.
52
-
53
- ## API stability
54
-
55
- Everything that is mentioned in this document can be considered stable in the
56
- sense that it will only change across major versions.
57
-
58
- There is one thing in this API documentation you can't and shouldn't rely on:
59
- The `SmellWarning` messages itself.
60
-
61
- Something like this
62
-
63
- ```
64
- Dirty#m has the parameter name 'a' (UncommunicativeParameterName)
65
- ```
66
-
67
- might change even across minor versions.
68
-
69
- You should not need to be specific about those messages anyways.
70
- In case you'd like to be specific about `SmellWarnings` please have a look at
71
- [accessing the smell warnings directly](#accessing-the-smell-warnings-directly).
72
-
73
- Additionally you can use one of our structured [outputs formats](#choosing-your-output-format)
74
- like JSON or YAML if you need a more fine-grained access to our
75
- `SmellWarnings`.
76
-
77
- ## Choosing your output format
78
-
79
- Besides normal text output, Reek can generate output in YAML,
80
- JSON, HTML and XML by using the following Report types:
81
-
82
- ```
83
- TextReport
84
- YAMLReport
85
- JSONReport
86
- HTMLReport
87
- XMLReport
88
- ```
89
-
90
- ## Configuration
91
-
92
- Given you have the following configuration file called `.reek.yml` in your root directory:
93
-
94
- ```Yaml
95
- ---
96
- IrresponsibleModule:
97
- enabled: false
98
- ```
99
-
100
- Reek will load this file automatically by default. If you want to load the
101
- configuration explicitely, you can use one of the methods below.
102
-
103
- You can now use either
104
-
105
- ```Ruby
106
- Reek::Configuration::AppConfiguration.from_path Pathname.new('config.reek')
107
- ```
108
-
109
- but you can also pass a hash with the contents of the `.reek.yml` YAML file
110
- to `Reek::Configuration::AppConfiguration.from_hash`.
111
-
112
- Given the example above you would load that as follows:
113
-
114
- ```Ruby
115
- require 'reek'
116
-
117
- config_hash = { 'IrresponsibleModule' => { 'enabled' => false } }
118
- configuration = Reek::Configuration::AppConfiguration.from_hash config_hash
119
-
120
- source = <<-RUBY
121
- class Dirty
122
- def call_me(a,b)
123
- puts a,b
124
- end
125
- end
126
- RUBY
127
-
128
- reporter = Reek::Report::TextReport.new
129
- examiner = Reek::Examiner.new(source, configuration: configuration); nil
130
- reporter.add_examiner examiner; nil
131
- reporter.show
132
- ```
133
-
134
- This would now only report `UncommunicativeParameterName` but not
135
- `IrresponsibleModule` for the `Dirty` class:
136
-
137
- ```
138
- string -- 2 warnings:
139
- Dirty#call_me has the parameter name 'a' (UncommunicativeParameterName)
140
- Dirty#call_me has the parameter name 'b' (UncommunicativeParameterName)
141
- ```
142
-
143
- Of course, directory specific configuration and excluded paths are supported as
144
- well:
145
-
146
- ```
147
- config_hash = {
148
- 'IrresponsibleModule' => { 'enabled' => false }
149
- 'spec/samples/three_clean_files/' =>
150
- { 'UtilityFunction' => { "enabled" => false } }
151
- 'exclude_paths' =>
152
- [ 'spec/samples/two_smelly_files' ]
153
- }
154
- ```
155
-
156
- ## Accessing the smell warnings directly
157
-
158
- You can also access the smells detected by an examiner directly:
159
-
160
- ```ruby
161
- require 'reek'
162
-
163
- source = <<-END
164
- class C
165
- end
166
- END
167
-
168
- examiner = Reek::Examiner.new source
169
- examiner.smells.each do |smell|
170
- puts smell.message
171
- end
172
- ```
173
-
174
- `Examiner#smells` returns a list of `SmellWarning` objects.
data/docs/Attribute.md DELETED
@@ -1,39 +0,0 @@
1
- # Attribute
2
-
3
- ## Introduction
4
-
5
- A class that publishes a setter for an instance variable invites
6
- client classes to become too intimate with its inner workings, and in
7
- particular with its representation of state.
8
-
9
- The same holds to a lesser extent for getters, but Reek doesn't flag those.
10
-
11
- ## Example
12
-
13
- Given:
14
-
15
- ```Ruby
16
- class Klass
17
- attr_accessor :dummy
18
- end
19
- ```
20
-
21
- Reek would emit the following warning:
22
-
23
- ```
24
- reek test.rb
25
-
26
- test.rb -- 1 warning:
27
- [2]:Attribute: Klass#dummy is a writable attribute
28
- ```
29
-
30
- ## Support in Reek
31
-
32
- This detector raises a warning for every public `attr_writer`,
33
- `attr_accessor`, and `attr` with the writable flag set to `true`.
34
-
35
- Reek does not raise warnings for read-only attributes.
36
-
37
- ## Configuration
38
-
39
- _Attribute_ supports only the [Basic Smell Options](Basic-Smell-Options.md).