leftovers 0.5.5 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0d697aa7197a87a0e3cc99124031957ab4cc602c89afd6d0369be3fffdef3ba0
-  data.tar.gz: c782e42dc2b5916160213fe22a7eb3c1560930c7e21f6f72489f10d0a03ff612
+  metadata.gz: 32e8f2762f7f513145ea799b64a059c7228dd7e9a4e08c2762abe92aad1370b1
+  data.tar.gz: 9134e7d5c66bcbb25bb5fba15f5b1bb661ce18972cf91e179f165af52f955662
 SHA512:
-  metadata.gz: c63edb11d4ed98ab23402f4a7ace623a6a364a5f39218076a5713034879fd98ed90fe024badaca2ac26562e94bbd0fde690f932e185de88a93bff1dca3fb0ce4
-  data.tar.gz: daae5206e9b217ce77201347c504255834aba552a8566c04ecfd224105577197c9d62415f7dc799727842c136d9e2c00e2fe90de8115c95fd4c176068cbcb313
+  metadata.gz: e361dc15e0a3496dadf81f91d356183631b1ad9f0b44951f976785622aa8550ccd0d8a6632064ab646a65231c4b772236cf1136b9267ceb81c59217d36bb4f78
+  data.tar.gz: d82fd190746370c9ead45a27e5a660f81aafadf58736afd644d5fafad816daa450f25eebcf30d18875b185ab7d76b3215a696ed35bb75ad33509a474cff87736

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+# 0.6.0
+- drop ruby 2.4 support, allowing for some performance improvements
+- Add ability to parse JSON and YAML files
+- Add magic comment that points to a particular dynamic rule
+- repeated calls to --write-todo won't have ordering differences
+- fix issue with --write-todo and unused methods defined in test files
+
 # 0.5.5
 - Fix rails resource/resources method signatures
 

data/README.md

@@ -129,7 +129,7 @@ To do this for all definitions of this name, instead of adding a comment, add th
 
 ### `# leftovers:call`
 _aliases `leftovers:calls`_
-To mark a dynamic call that doesn't use literal values, use `leftovers:call` with the method name listed
+To mark a call that doesn't use literal values, use `leftovers:call` with the method name listed
 ```ruby
 method = [:puts, :warn].sample # leftovers:call puts, warn
 send(method, 'text')
@@ -137,6 +137,26 @@ send(method, 'text')
 
 This would consider `puts` and `warn` to both have been called
 
+### `# leftovers:dynamic:*`
+To mark a dynamic call for literal hash and array values without enumerating everything in the comment again, use `leftovers:dynamic:`  on the same line as the beginning of the array or hash
+
+```ruby
+[ # leftovers:dynamic:call_login
+  :user,
+  :admin
+].each { |method| send("#{method}_login") }
+```
+
+with the following configuration matching the `name: value` to the `leftovers:dynamic:process_name`
+
+```yaml
+dynamic:
+  name: call_login
+  arguments: '*'
+  add_suffix: '_login'
+```
+This would consider `user_login` and `admin_login` to both have been called.
+
 ## Configuration file
 
 The configuration is read from `.leftovers.yml` in your project root.

data/docs/Configuration.md

@@ -9,6 +9,8 @@ Its presence is optional and all of these settings are optional.
 - [`haml_paths:`](#haml_paths)
 - [`slim_paths:`](#slim_paths)
 - [`erb_paths:`](#erb_paths)
+- [`yaml_paths:`](#yaml_paths)
+- [`json_paths:`](#json_paths)
 - [`requires:`](#requires)
 - [`gems:`](#gems)
 - [`keep:`](#keep)
@@ -117,6 +119,76 @@ erb_paths:
 
 Arrays are not necessary for single values. `*.erb` is recognized by default
 
+## `yaml_paths:`
+
+list filenames/paths of test directories that are in the yaml format
+Defined using the [.gitignore pattern format](https://git-scm.com/docs/gitignore#_pattern_format)
+
+```yml
+include:
+  - 'config/*.yml'
+yaml_paths:
+  - '*.yml'
+```
+
+These documents will consider yaml tags like `!ruby/class 'MyClass'` to be a call to `MyClass` and render the structure of the yaml as arguments for the [`document:true`](#document-true) rule.
+
+so you could, e.g. read the class name out of a yaml document like:
+
+```yml
+class_name: MyClass
+```
+
+with config like:
+
+```yml
+include_paths:
+  - 'config/*.yml'
+
+dynamic:
+  document: true
+  path: config/*.yml
+  calls:
+    argument: class_name
+```
+
+[`nested:`](#nested) may be useful for more complex yaml structures
+
+## `json_paths:`
+
+list filenames/paths of test directories that are in the json format
+Defined using the [.gitignore pattern format](https://git-scm.com/docs/gitignore#_pattern_format)
+
+```yml
+include:
+  - 'config/*.json'
+json_paths:
+  - '*.json'
+```
+
+These documents render the structure of the yaml as arguments for the [`document:true`](#document-true) rule.
+
+so you could, e.g. read the class name out of a yaml document like:
+
+```json
+{ "class_name": "MyClass" }
+```
+
+with config like:
+
+```yml
+include_paths:
+  - 'config/*.json'
+
+dynamic:
+  document: true
+  path: config/*.json
+  calls:
+    argument: class_name
+```
+
+[`nested:`](#nested) may be useful for more complex json structures
+
 ## `gems:`
 _alias `gem:`_
 
@@ -202,7 +274,8 @@ Each entry must have at least one of the following properties to restrict which
 - [`paths:`](#paths)
 - [`has_arguments:`](#has_arguments)
 - [`has_receiver:`](#has_receiver)
-- [`unless`](#unless)
+- [`unless:`](#unless)
+- [`document: true`](#document-true)
 
 And must have one or both of
 - ['calls:`](#calls-defines)
@@ -299,6 +372,33 @@ keep:
     path: /app/helpers
 ```
 
+## `document: true`
+
+Instructs to consider the whole document. this is useful when parsing [YAML](#yaml-paths) or [JSON](#json-paths) config files for various values.
+
+e.g.
+
+```yml
+includes: /config/roles.yml
+dynamic:
+  - document: true
+    path: /config/roles.yml
+    defines:
+      arguments: '*'
+      add_suffix: '?'
+      add_prefix: can_
+```
+
+will parse "config/roles.yml"
+```yml
+- build_house
+- drive_car
+```
+
+and consider it to have created methods like `can_build_house?` and `can_drive_car?`
+
+[`nested:`](#nested) may be useful for more complex data structures
+
 ## `calls:`, `defines:`
 _aliases `call:`, `define:`_
 
@@ -366,7 +466,9 @@ It can have any of these properties:
 - [`at:`](#at)
 - [`has_value:`](#has_value_has_receiver)
 
-Arrays are not necessary for single values and if the rule contains only `at:` it can be omitted, and the values moved up a level
+Arrays are not necessary for single values and if the rule contains only `at:` it can be omitted, and the values moved up a level.
+
+Positional arguments are zero indexed
 
 ## `has_arguments:`
 _alias `has_argument:`_
@@ -384,6 +486,8 @@ It can have any of these properties:
 
 Arrays are not necessary for single values and if the rule contains only `at:` it can be omitted, and the values moved up a level
 
+Positional arguments are zero indexed
+
 ## `keywords:`
 When the keyword argument **keywords** are the thing being called.
 

data/leftovers.gemspec

@@ -17,7 +17,9 @@ Gem::Specification.new do |spec|
   spec.metadata['homepage_uri'] = spec.homepage
   spec.metadata['source_code_uri'] = 'http://github.com/robotdana/leftovers'
   spec.metadata['changelog_uri'] = 'http://github.com/robotdana/leftovers/blob/main/CHANGELOG.md'
-  spec.required_ruby_version = '>= 2.4.0'
+
+  spec.metadata['rubygems_mfa_required'] = 'true'
+  spec.required_ruby_version = '>= 2.5.0'
 
   spec.files = Dir.glob('{lib,exe,docs}/**/{*,.*}') + %w{
     CHANGELOG.md
@@ -37,9 +39,10 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'pry', '~> 0.1'
   spec.add_development_dependency 'rake', '>= 13'
   spec.add_development_dependency 'rspec', '~> 3.0'
-  spec.add_development_dependency 'rubocop', '~> 0.93.1'
-  spec.add_development_dependency 'rubocop-performance', '~> 1.8.1'
-  spec.add_development_dependency 'rubocop-rspec', '~> 1.44.1'
+  spec.add_development_dependency 'rubocop'
+  spec.add_development_dependency 'rubocop-performance'
+  spec.add_development_dependency 'rubocop-rake'
+  spec.add_development_dependency 'rubocop-rspec'
   spec.add_development_dependency 'simplecov', '>= 0.18.5'
   spec.add_development_dependency 'simplecov-console'
   spec.add_development_dependency 'slim'

data/lib/config/ruby.yml

@@ -10,6 +10,13 @@ include_paths:
   - '*.erb'
   - '*.rb'
 
+yaml_paths:
+  - '*.yml'
+  - '*.yaml'
+
+json_paths:
+  - '*.json'
+
 test_paths:
   - /tests/
   - /test/

data/lib/leftovers/collector.rb

@@ -28,7 +28,9 @@ module Leftovers
 
     def collect_file_list(list)
       if Leftovers.parallel?
-        Parallel.each(list, finish: method(:finish_file), &method(:collect_file))
+        Parallel.each(list, finish: method(:finish_file)) do |file|
+          collect_file(file)
+        end
       else
         list.each { |file| finish_file(nil, nil, collect_file(file)) }
       end

data/lib/leftovers/config.rb

@@ -4,10 +4,6 @@ require 'yaml'
 
 module Leftovers
   class Config
-    # :nocov:
-    using ::Leftovers::Backports::SetCaseEq if defined?(::Leftovers::Backports::SetCaseEq)
-    # :nocov:
-
     attr_reader :name
 
     def initialize(name, path: nil, content: nil)
@@ -40,6 +36,14 @@ module Leftovers
       @slim_paths ||= Array(yaml[:slim_paths])
     end
 
+    def yaml_paths
+      @yaml_paths ||= Array(yaml[:yaml_paths])
+    end
+
+    def json_paths
+      @json_paths ||= Array(yaml[:json_paths])
+    end
+
     def erb_paths
       @erb_paths ||= Array(yaml[:erb_paths])
     end