leftovers 0.5.3 → 0.5.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8bd5a1759f2b1f26facd015571005bb0bbc3a015ad6feeb28a5b79ec368edf73
-  data.tar.gz: dbebde9858cd4566a8ca0b2d3770ae97d7904934fe735cce07a287a1c99c55a3
+  metadata.gz: cc87c83e3671055ddbbd445768b73d55576676da6cfff8d83f5ceb57eb1d78e8
+  data.tar.gz: a3cc4945b1d4158817d0a6b5a0c0cc8bc8ad1d2e7a402d0b0a3221cebba7dc02
 SHA512:
-  metadata.gz: 5e81ea69d2763105ba675d36a60c09b6274c625426f92088c2b560cf2890dffb714d56574af91dc51bbdaa64c6e419fff497415a847dc86c239fc19db3c4b026
-  data.tar.gz: db7bf18379a515c2b53bcc912b7e97e7b44e8bbb75ee1f76db2c729f08f6617852df69c0d7dce4a16d284b31026d5d8da07ec1ff5c655693bf0dfc9ea4ebc523
+  metadata.gz: 450162ef45051251a698a5cfbd0451ac50d175339b2a9888edd7790e29071a4360f7677ca4f8d6b927bfdc004b0e4fde747884804c8e480659b76a5b39d0f8e3
+  data.tar.gz: 6ae0474561c71a1eddd0f6ecb0ba184de668eaca4aea36da0460fbeec313f9f095533f913863913de89ad192903b1cadaaffb3231432136138b2720df094f636

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+# 0.5.4
+- Add support for slim templates #13 - thanks @veganstraightedge
+- fix the #how-to-resolve link #11 - thanks @veganstraightedge
+
 # v0.5.3
 - fix incompatibility with activesupport 7.
 

data/README.md

@@ -57,7 +57,7 @@ Not directly called at all:
 lib/hello_world.rb:6:6 generated_method= attr_accessor :generated_method
 lib/hello_world.rb:6:6 generated_method attr_accessor :generated_method
 
-how to resolve: https://github.com/robotdana/leftovers/tree/main/Readme.md#how_to_resolve
+how to resolve: https://github.com/robotdana/leftovers/tree/main/Readme.md#how-to-resolve
 ```
 
 if there is an overwhelming number of results, try using [`--write-todo`](#write-todo)
@@ -163,7 +163,7 @@ see the [built in config files](https://github.com/robotdana/leftovers/tree/main
 
   Add the method/pattern to the `dynamic:` list with `skip: true` in the `.leftovers.yml`, or add an inline comment with the list of possibilities `# leftovers:call my_method_1, my_method_2`.
 - Leftovers compares by name only, so multiple definitions with the same name will count as used even if only one is.
-- haml & erb line and column numbers will be wrong as the files have to be precompiled before checking.
+- haml, slim & erb line and column numbers will be wrong as the files have to be precompiled before checking.
 
 ## Other tools
 

data/docs/Configuration.md

@@ -7,6 +7,7 @@ Its presence is optional and all of these settings are optional.
 - [`exclude_paths:`](#exclude_paths)
 - [`test_paths:`](#test_paths)
 - [`haml_paths:`](#haml_paths)
+- [`slim_paths:`](#slim_paths)
 - [`erb_paths:`](#erb_paths)
 - [`requires:`](#requires)
 - [`gems:`](#gems)
@@ -92,6 +93,18 @@ haml_paths:
 
 Arrays are not necessary for single values. `*.haml` is recognized by default
 
+## `slim_paths:`
+
+list filenames/paths of test directories that are in the slim format
+Defined using the [.gitignore pattern format](https://git-scm.com/docs/gitignore#_pattern_format)
+
+```yml
+slim_paths:
+  - '*.slim'
+```
+
+Arrays are not necessary for single values. `*.slim` is recognized by default
+
 ## `erb_paths:`
 
 list filenames/paths of test directories that are in the erb format

data/leftovers.gemspec

@@ -42,6 +42,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rubocop-rspec', '~> 1.44.1'
   spec.add_development_dependency 'simplecov', '>= 0.18.5'
   spec.add_development_dependency 'simplecov-console'
+  spec.add_development_dependency 'slim'
   spec.add_development_dependency 'timecop'
   spec.add_development_dependency 'tty_string', '>= 0.2.1'
 

data/lib/config/ruby.yml

@@ -20,6 +20,9 @@ erb_paths:
 haml_paths:
   - '*.haml'
 
+slim_paths:
+  - '*.slim'
+
 keep:
   - initialize # called by new
   - inspect # called by repl, to_s

data/lib/config/slim.yml

@@ -0,0 +1,4 @@
+include_paths:
+  - '*.slim'
+slim_paths:
+  - '*.slim'

data/lib/leftovers/config.rb

@@ -36,6 +36,10 @@ module Leftovers
       @haml_paths ||= Array(yaml[:haml_paths])
     end
 
+    def slim_paths
+      @slim_paths ||= Array(yaml[:slim_paths])
+    end
+
     def erb_paths
       @erb_paths ||= Array(yaml[:erb_paths])
     end

data/lib/leftovers/config_validator/schema_hash.rb

@@ -517,6 +517,7 @@ module Leftovers
         'exclude_paths' => { '$ref' => '#/definitions/stringList' },
         'test_paths' => { '$ref' => '#/definitions/stringList' },
         'haml_paths' => { '$ref' => '#/definitions/stringList' },
+        'slim_paths' => { '$ref' => '#/definitions/stringList' },
         'erb_paths' => { '$ref' => '#/definitions/stringList' },
         'requires' => { '$ref' => '#/definitions/stringList' },
         'gems' => { '$ref' => '#/definitions/stringList' },

data/lib/leftovers/erb.rb

@@ -4,7 +4,7 @@ require 'erb'
 
 module Leftovers
   class ERB < ::ERB::Compiler
-    def self.precompile(erb)
+    def self.precompile(erb, _name)
       @compiler ||= new('-')
       @compiler.compile(erb).first
     end

data/lib/leftovers/file.rb

@@ -15,12 +15,18 @@ module Leftovers
     end
 
     def ruby
+      precompiler&.precompile(read, self) || read
+    end
+
+    private
+
+    def precompiler
       if Leftovers.config.haml_paths.allowed?(relative_path)
-        ::Leftovers::Haml.precompile(read, self)
+        ::Leftovers::Haml
+      elsif Leftovers.config.slim_paths.allowed?(relative_path)
+        ::Leftovers::Slim
       elsif Leftovers.config.erb_paths.allowed?(relative_path)
-        ::Leftovers::ERB.precompile(read)
-      else
-        read
+        ::Leftovers::ERB
       end
     end
   end

data/lib/leftovers/merged_config.rb

@@ -37,14 +37,22 @@ module Leftovers
       Leftovers::Config.new(:'.leftovers_todo.yml', path: Leftovers.pwd + '.leftovers_todo.yml')
     end
 
-    def unmemoize # rubocop:disable Metrics/CyclomaticComplexity
-      remove_instance_variable(:@exclude_paths) if defined?(@exclude_paths)
-      remove_instance_variable(:@include_paths) if defined?(@include_paths)
-      remove_instance_variable(:@test_paths) if defined?(@test_paths)
-      remove_instance_variable(:@haml_paths) if defined?(@haml_paths)
-      remove_instance_variable(:@erb_paths) if defined?(@erb_paths)
-      remove_instance_variable(:@dynamic) if defined?(@dynamic)
-      remove_instance_variable(:@keep) if defined?(@keep)
+    MEMOIZED_IVARS = %i{
+      @exclude_paths
+      @include_paths
+      @test_paths
+      @haml_paths
+      @slim_paths
+      @erb_paths
+      @dynamic
+      @keep
+      @test_only
+    }.freeze
+
+    def unmemoize
+      MEMOIZED_IVARS.each do |ivar|
+        remove_instance_variable(ivar) if instance_variable_get(ivar)
+      end
     end
 
     def exclude_paths
@@ -71,6 +79,14 @@ module Leftovers
       )
     end
 
+    def slim_paths
+      @slim_paths ||= FastIgnore.new(
+        include_rules: @configs.flat_map(&:slim_paths),
+        gitignore: false,
+        root: Leftovers.pwd
+      )
+    end
+
     def erb_paths
       @erb_paths ||= FastIgnore.new(
         include_rules: @configs.flat_map(&:erb_paths),

data/lib/leftovers/slim.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Leftovers
+  module Slim
+    module_function
+
+    def precompile(file, name)
+      return '' unless Leftovers.try_require('slim', message: <<~MESSAGE) # rubocop:disable Layout/EmptyLineAfterGuardClause
+        Skipped parsing #{name.relative_path}, because the slim gem was not available
+        `gem install slim`
+      MESSAGE
+
+      begin
+        ::Slim::Engine.new(file: file).call(file)
+      rescue ::Slim::Parser::SyntaxError => e
+        Leftovers.warn "#{e.class}: \"#{e.error}\" #{name.relative_path}:#{e.lineno}:#{e.column}"
+        ''
+      end
+    end
+  end
+end

data/lib/leftovers/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Leftovers
-  VERSION = '0.5.3'
+  VERSION = '0.5.4'
 end

data/lib/leftovers.rb

@@ -24,6 +24,7 @@ module Leftovers # rubocop:disable Metrics/ModuleLength
   autoload(:ProcessorBuilders, "#{__dir__}/leftovers/processor_builders")
   autoload(:RakeTask, "#{__dir__}/leftovers/rake_task")
   autoload(:Reporter, "#{__dir__}/leftovers/reporter")
+  autoload(:Slim, "#{__dir__}/leftovers/slim")
   autoload(:TodoReporter, "#{__dir__}/leftovers/todo_reporter")
   autoload(:DynamicProcessors, "#{__dir__}/leftovers/dynamic_processors")
   autoload(:ValueProcessors, "#{__dir__}/leftovers/value_processors")
@@ -93,7 +94,7 @@ module Leftovers # rubocop:disable Metrics/ModuleLength
     end
 
     def resolution_instructions_link
-      "https://github.com/robotdana/leftovers/tree/v#{Leftovers::VERSION}/README.md#how_to_resolve"
+      "https://github.com/robotdana/leftovers/tree/v#{Leftovers::VERSION}/README.md#how-to-resolve"
     end
 
     def warn(message)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: leftovers
 version: !ruby/object:Gem::Version
-  version: 0.5.3
+  version: 0.5.4
 platform: ruby
 authors:
 - Dana Sherson
@@ -178,6 +178,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: slim
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: timecop
   requirement: !ruby/object:Gem::Requirement
@@ -323,6 +337,7 @@ files:
 - lib/config/selenium-webdriver.yml
 - lib/config/sidekiq.yml
 - lib/config/simplecov.yml
+- lib/config/slim.yml
 - lib/config/will_paginate.yml
 - lib/leftovers.rb
 - lib/leftovers/ast.rb
@@ -407,6 +422,7 @@ files:
 - lib/leftovers/processor_builders/value.rb
 - lib/leftovers/rake_task.rb
 - lib/leftovers/reporter.rb
+- lib/leftovers/slim.rb
 - lib/leftovers/todo_reporter.rb
 - lib/leftovers/value_processors.rb
 - lib/leftovers/value_processors/add_dynamic_prefix.rb