haml_lint 0.52.0 → 0.53.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 24f37fc2b314359fb58b5209e58399fa01f53d51c977a4c60b66d9497e2acde5
-  data.tar.gz: afb033a6021882a269ec1fdf989d06c83c6df8047a3f60233ac83ac9f234469d
+  metadata.gz: 49bb0bec15e96c047439560e80f83eaf6f4c3baec24a12b1f10248f1d2f347d3
+  data.tar.gz: 153f65eda76c5751bbae1d02fc49d0d4477f401bb273096ad9308868b9d21288
 SHA512:
-  metadata.gz: abb4758509d1aca40826e598c8d1bdfb7daffbbbeb9d1955c18463e44c4799ce5102151ae34b58799198336d87ba2c4e922a699b8f9c28b13f94479d80f4607a
-  data.tar.gz: f585fb701da7d6ca46ab9612055a25be83c640c97da530fa19ca20684e27056955ca80714a666fef1568fb8fa4bceba5830005167b3f4a76dc575732876d125e
+  metadata.gz: a4d2c2682e7b12d9aaf8164054f3533c0667f5f6035bfbf5ef6bbb694a40f9a70d9d13a9aa56dabf27131be27e74d346c9eff642a65d0e2aac8992ebc1999fba
+  data.tar.gz: 343aee9ed9bf4a766640aabbbaf96af9a5c36261fb6e6943b9a0b9eadcb5c75bc5a15665cae30c3c4c8925b307ecca837130fb04cf1d14a74549978db235f60c

data/lib/haml_lint/adapter.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require 'haml_lint/adapter/haml_4'
 require 'haml_lint/adapter/haml_5'
 require 'haml_lint/adapter/haml_6'
 require 'haml_lint/exceptions'
@@ -19,7 +18,6 @@ module HamlLint
     def self.detect_class
       version = haml_version
       case version
-      when '~> 4.0' then HamlLint::Adapter::Haml4
       when '~> 5.0', '~> 5.1', '~> 5.2' then HamlLint::Adapter::Haml5
       when '~> 6.0', '~> 6.0.a', '~> 6.1', '~> 6.2', '~> 6.3' then HamlLint::Adapter::Haml6
       else fail HamlLint::Exceptions::UnknownHamlVersion, "Cannot handle Haml version: #{version}"

data/lib/haml_lint/ruby_extraction/chunk_extractor.rb

@@ -84,7 +84,35 @@ module HamlLint::RubyExtraction
     def visit_comment(node)
       line = @original_haml_lines[node.line - 1]
       indent = line.index(/\S/)
+
       @ruby_chunks << PlaceholderMarkerChunk.new(node, 'comment', indent: indent)
+
+      # Comment can have subnodes, such as plain. This happens for conditional comments, such as:
+      # %head
+      #   /[if mso]
+      #     %div
+      if node.children
+        # We don't want to use a block because assignments in a block are local to that block,
+        # so the semantics of the extracted ruby would be different from the one generated by
+        # Haml. Those differences can make some cops, such as UselessAssignment, have false
+        # positives
+        begin_chunk = AdHocChunk.new(node, [' ' * indent + 'begin'])
+        @ruby_chunks << begin_chunk
+        indent += 2
+
+        yield
+
+        indent -= 2
+
+        if @ruby_chunks.last.equal?(begin_chunk)
+          # So there is nothing nesting, remove the wrapping "begin"
+          @ruby_chunks.pop
+        else
+          @ruby_chunks << AdHocChunk.new(node,
+                                         [' ' * indent + 'ensure', ' ' * indent + '  HL.noop', ' ' * indent + 'end'],
+                                         haml_line_index: @ruby_chunks.last.haml_end_line_index)
+        end
+      end
     end
 
     # Visit a script which outputs. Lines looking like `  = foo`

data/lib/haml_lint/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: haml_lint
 version: !ruby/object:Gem::Version
-  version: 0.52.0
+  version: 0.53.0
 platform: ruby
 authors:
 - Shane da Silva
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-12-13 00:00:00.000000000 Z
+date: 2024-01-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: haml
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: parallel
   requirement: !ruby/object:Gem::Requirement
@@ -93,7 +93,6 @@ files:
 - config/forced_rubocop_config.yml
 - lib/haml_lint.rb
 - lib/haml_lint/adapter.rb
-- lib/haml_lint/adapter/haml_4.rb
 - lib/haml_lint/adapter/haml_5.rb
 - lib/haml_lint/adapter/haml_6.rb
 - lib/haml_lint/cli.rb
@@ -200,7 +199,7 @@ homepage: https://github.com/sds/haml-lint
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -215,8 +214,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key:
+rubygems_version: 3.0.3.1
+signing_key: 
 specification_version: 4
 summary: HAML lint tool
 test_files: []

data/lib/haml_lint/adapter/haml_4.rb

@@ -1,62 +0,0 @@
-# frozen_string_literal: true
-
-require 'forwardable'
-
-module HamlLint
-  class Adapter
-    # Adapts the Haml::Parser from Haml 4 for use in HamlLint
-    # :reek:UncommunicativeModuleName
-    class Haml4 < Adapter
-      extend Forwardable
-
-      # Parses the specified Haml code into an abstract syntax tree
-      #
-      # @example
-      #   HamlLint::Adapter::Haml4.new('%div')
-      #
-      # @api public
-      # @param source [String] Haml code to parse
-      # @param options [Haml::Options]
-      def initialize(source, options = Haml::Options.new)
-        @source = source
-        @parser = Haml::Parser.new(source, options)
-      end
-
-      def precompile
-        # Haml uses the filters as part of precompilation... we don't care about those,
-        # but without this tweak, it would fail on filters that are not loaded.
-        real_defined = Haml::Filters.defined
-        Haml::Filters.instance_variable_set(:@defined, Hash.new { real_defined['plain'] })
-
-        ::Haml::Engine.new(source).precompiled
-      ensure
-        Haml::Filters.instance_variable_set(:@defined, real_defined)
-      end
-
-      # @!method
-      #   Parses the source code into an abstract syntax tree
-      #
-      #   @example
-      #     HamlLint::Adapter::Haml4.new('%div')
-      #
-      #   @api public
-      #   @return [Haml::Parser::ParseNode]
-      # @raise [Haml::Error]
-      def_delegator :parser, :parse
-
-      private
-
-      # The Haml parser to adapt for HamlLint
-      #
-      # @api private
-      # @return [Haml::Parser] the Haml 4 parser
-      attr_reader :parser
-
-      # The Haml code to parse
-      #
-      # @api private
-      # @return [String] Haml code to parse
-      attr_reader :source
-    end
-  end
-end