haml_lint 0.34.1 → 0.37.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2e13afbca153d2eb77c44ea3c266bf1a970ef71aa5d8efaa3c956870f69d76e0
-  data.tar.gz: 7b48e11ccc6bec1d6750c5905df9b05b48980163a87c2b00dad45a6364ef35df
+  metadata.gz: 8863f65bfdd024cd8dcb34e0c0f76401798afc1ba86e3f68a1bf128fad4d7a0e
+  data.tar.gz: c659429275e92bbc87200848bd616c73cf1a9a418ea8c5bc042425cbb0c34dae
 SHA512:
-  metadata.gz: 17cb994e2f2498745c02b2c584f23b03a81a67fa4f6fb4d1f1b2d9421e4d2426fbe04c39d1d49288d10bf14c7b0dde2d2f2c2dbc991a5e7839f6085438d4984b
-  data.tar.gz: 553834970edaffca09efba96a422ee73d45251a5c48d1e15a01a3cf0184d2d8e90392cd663d4f836e061a7486b061998859032b090c6916a968b66019ac9b581
+  metadata.gz: 9281eed257dc6972cbe29fe8e9d1f00affd6f1d8bb6880ae4e10a6252722e65feaac262a148b94ed25ebfd401e4ec17403169ea5a0613a0fa56921cbcb34585f
+  data.tar.gz: 8bd38dfe3c05da8ce82d7a021c22eba361af40d53d07a38b55c2b4452e05fcc3efce99f556f7725a51b21f66b9f36993f77ebc3596cd7bd367e89054b4fafd06

data/config/default.yml

@@ -93,11 +93,13 @@ linters:
       - Lint/Void
       - Layout/AlignHash # renamed to Layout/HashAlignment in rubocop 0.77
       - Layout/AlignParameters # renamed to Layout/ParameterAlignment in rubocop 0.77
+      - Layout/ArgumentAlignment
       - Layout/CaseIndentation
       - Layout/ElseAlignment
       - Layout/EndOfLine
       - Layout/HashAlignment
       - Layout/IndentationWidth
+      - Layout/LineLength # renamed from Metrics/LineLength in rubocop 0.79.0
       - Layout/ParameterAlignment
       - Layout/TrailingBlankLines # renamed to Layout/TrailingEmptyLines in rubocop 0.77
       - Layout/TrailingEmptyLines

data/lib/haml_lint.rb

@@ -26,16 +26,16 @@ require 'haml_lint/severity'
 # Load all parse tree node classes
 require 'haml_lint/tree/node'
 require 'haml_lint/node_transformer'
-Dir[File.expand_path('haml_lint/tree/*.rb', File.dirname(__FILE__))].each do |file|
+Dir[File.expand_path('haml_lint/tree/*.rb', File.dirname(__FILE__))].sort.each do |file|
   require file
 end
 
 # Load all linters
-Dir[File.expand_path('haml_lint/linter/*.rb', File.dirname(__FILE__))].each do |file|
+Dir[File.expand_path('haml_lint/linter/*.rb', File.dirname(__FILE__))].sort.each do |file|
   require file
 end
 
 # Load all reporters
-Dir[File.expand_path('haml_lint/reporter/*.rb', File.dirname(__FILE__))].each do |file|
+Dir[File.expand_path('haml_lint/reporter/*.rb', File.dirname(__FILE__))].sort.each do |file|
   require file
 end

data/lib/haml_lint/adapter.rb

@@ -19,7 +19,7 @@ module HamlLint
       version = haml_version
       case version
       when '~> 4.0' then HamlLint::Adapter::Haml4
-      when '~> 5.0', '~> 5.1' then HamlLint::Adapter::Haml5
+      when '~> 5.0', '~> 5.1', '~> 5.2' then HamlLint::Adapter::Haml5
       else fail HamlLint::Exceptions::UnknownHamlVersion, "Cannot handle Haml version: #{version}"
       end
     end

data/lib/haml_lint/cli.rb

@@ -94,7 +94,7 @@ module HamlLint
     # @return [HamlLint::Reporter]
     def reporter_from_options(options)
       if options[:auto_gen_config]
-        HamlLint::Reporter::DisabledConfigReporter.new(log, limit: options[:auto_gen_exclude_limit] || 15) # rubocop:disable Metrics/LineLength
+        HamlLint::Reporter::DisabledConfigReporter.new(log, limit: options[:auto_gen_exclude_limit] || 15) # rubocop:disable Layout/LineLength
       else
         options.fetch(:reporter, HamlLint::Reporter::DefaultReporter).new(log)
       end

data/lib/haml_lint/linter/rubocop.rb

@@ -29,6 +29,14 @@ module HamlLint
 
     private
 
+    # A single CLI instance is shared between files to avoid RuboCop
+    # having to repeatedly reload .rubocop.yml.
+    def self.rubocop_cli
+      # The ivar is stored on the class singleton rather than the Linter instance
+      # because it can't be Marshal.dump'd (as used by Parallel.map)
+      @rubocop_cli ||= ::RuboCop::CLI.new
+    end
+
     # Executes RuboCop against the given Ruby code and records the offenses as
     # lints.
     #
@@ -36,8 +44,6 @@ module HamlLint
     # @param source_map [Hash] map of Ruby code line numbers to original line
     #   numbers in the template
     def find_lints(ruby, source_map)
-      rubocop = ::RuboCop::CLI.new
-
       filename =
         if document.file
           "#{document.file}.rb"
@@ -46,7 +52,7 @@ module HamlLint
         end
 
       with_ruby_from_stdin(ruby) do
-        extract_lints_from_offenses(lint_file(rubocop, filename), source_map)
+        extract_lints_from_offenses(lint_file(self.class.rubocop_cli, filename), source_map)
       end
     end
 
@@ -56,7 +62,12 @@ module HamlLint
     # @param file [String]
     # @return [Array<RuboCop::Cop::Offense>]
     def lint_file(rubocop, file)
-      rubocop.run(rubocop_flags << file)
+      status = rubocop.run(rubocop_flags << file)
+      unless [::RuboCop::CLI::STATUS_SUCCESS, ::RuboCop::CLI::STATUS_OFFENSES].include?(status)
+        raise HamlLint::Exceptions::ConfigurationError,
+              "RuboCop exited unsuccessfully with status #{status}." \
+              ' Check the stack trace to see if there was a misconfiguration.'
+      end
       OffenseCollector.offenses
     end
 

data/lib/haml_lint/linter/unnecessary_string_output.rb

@@ -43,7 +43,7 @@ module HamlLint
     # given special treatment, thus making enclosing it in a string necessary.
     def starts_with_reserved_character?(stringish)
       string = stringish.respond_to?(:children) ? stringish.children.first : stringish
-      string =~ %r{\A\s*[/#-=%~]}
+      string =~ %r{\A\s*[/#-=%~]} if string.is_a?(String)
     end
   end
 end

data/lib/haml_lint/options.rb

@@ -38,7 +38,7 @@ module HamlLint
       end
 
       parser.on('--auto-gen-exclude-limit limit', Integer,
-                'Number of failures to allow in the TODO list before the entire rule is excluded') do |limit| # rubocop:disable Metrics/LineLength
+                'Number of failures to allow in the TODO list before the entire rule is excluded') do |limit| # rubocop:disable Layout/LineLength
         @options[:auto_gen_exclude_limit] = limit
       end
 
@@ -51,6 +51,10 @@ module HamlLint
                 "Specify which linters you don't want to run") do |linters|
         @options[:excluded_linters] = linters
       end
+
+      parser.on('-p', '--parallel', 'Run linters in parallel using available CPUs') do
+        @options[:parallel] = true
+      end
     end
 
     def add_report_options(parser)

data/lib/haml_lint/runner.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'parallel'
+
 module HamlLint
   # Responsible for running the applicable linters against the desired files.
   class Runner
@@ -21,6 +23,7 @@ module HamlLint
       @files = extract_applicable_files(config, options)
       @linter_selector = HamlLint::LinterSelector.new(config, options)
       @fail_fast = options.fetch(:fail_fast, false)
+      @cache = {}
 
       report(options)
     end
@@ -121,7 +124,7 @@ module HamlLint
     # @param report [HamlLint::Report]
     # @return [void]
     def process_file(file, report)
-      lints = collect_lints(file, linter_selector, config)
+      lints = @cache[file] || collect_lints(file, linter_selector, config)
       lints.each { |lint| report.add_lint(lint) }
       report.finish_file(file, lints)
     end
@@ -134,8 +137,20 @@ module HamlLint
     def report(options)
       report = HamlLint::Report.new(reporter: options[:reporter], fail_level: options[:fail_level])
       report.start(@files)
+      warm_cache if options[:parallel]
       process_files(report)
       report
     end
+
+    # Cache the result of processing lints in parallel.
+    #
+    # @return [void]
+    def warm_cache
+      results = Parallel.map(files) do |file|
+        lints = collect_lints(file, linter_selector, config)
+        [file, lints]
+      end
+      @cache = results.to_h
+    end
   end
 end

data/lib/haml_lint/tree/root_node.rb

@@ -17,7 +17,9 @@ module HamlLint::Tree
     # @param line [Integer] the line number of the node
     # @return [HamlLint::Node]
     def node_for_line(line)
-      find(-> { HamlLint::Tree::NullNode.new }) { |node| node.line_numbers.cover?(line) }
+      find(-> { HamlLint::Tree::NullNode.new }) do |node|
+        node.line_numbers.cover?(line) && node != self
+      end
     end
   end
 end

data/lib/haml_lint/tree/tag_node.rb

@@ -2,7 +2,7 @@
 
 module HamlLint::Tree
   # Represents a tag node in a HAML document.
-  class TagNode < Node # rubocop:disable ClassLength
+  class TagNode < Node # rubocop:disable Metrics/ClassLength
     # Computed set of attribute hashes code.
     #
     # This is a combination of all dynamically calculated attributes from the

data/lib/haml_lint/version.rb

@@ -2,5 +2,5 @@
 
 # Defines the gem version.
 module HamlLint
-  VERSION = '0.34.1'
+  VERSION = '0.37.1'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: haml_lint
 version: !ruby/object:Gem::Version
-  version: 0.34.1
+  version: 0.37.1
 platform: ruby
 authors:
 - Shane da Silva
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-12-10 00:00:00.000000000 Z
+date: 2021-06-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: haml
@@ -19,7 +19,7 @@ dependencies:
         version: '4.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '5.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -29,7 +29,21 @@ dependencies:
         version: '4.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '5.3'
+- !ruby/object:Gem::Dependency
+  name: parallel
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.10'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.10'
 - !ruby/object:Gem::Dependency
   name: rainbow
   requirement: !ruby/object:Gem::Requirement
@@ -171,7 +185,7 @@ homepage: https://github.com/sds/haml-lint
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -186,8 +200,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: HAML lint tool
 test_files: []