erb_lint 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5c54cf59770c0719d8cae0376d27a685ba2159d0e8884c00a472b0217d1ca2e6
-  data.tar.gz: c644e23a805c04e02daf4218f8548a167a90cc8d0b2bf0fd73acbbdbbaec237c
+  metadata.gz: b98507494f8af2a33e0e6fcbf28c48a2d91c78f42273a3cc6c1206578f3a55b0
+  data.tar.gz: c090a633a9d041a6d4d71bc292b1525b3a74dccd61311e0224bc1cbb0d8fea33
 SHA512:
-  metadata.gz: '07718e5eed2bfb8246a529615c3950645c0d51e376475c6cea7eb60bedfc17af42c4f24f3aba7e8184e842087027b3c0e033afb6c1d01eceb5b84e7c0da8bb74'
-  data.tar.gz: 6bda3b3ad2b03a0812a6bb796c0f6f2d03545fcdc44a6130dc7e3b2135d523e8decb395e3fd7599d9121045c840585de3c4ced6c9c8fb642d1865d263e978dad
+  metadata.gz: b2ac77d88b7be36010d69c072e3ee72c370edefa1fc3b68135593b5b494da8b0f9eac79e9b5408d10155a007d94948ae6a4b3b279de2541be66f0576abc964a8
+  data.tar.gz: f0be6a75c5060b49a8b7c5e5a5d19adb3222fb580a11c2e2e488e5b83ada00db0c4daee25e99ba5538057c749f5937b80d25a96bda1ed967e6d1baeddf787965

data/exe/erblint

@@ -3,7 +3,7 @@
 
 $LOAD_PATH.unshift("#{__dir__}/../lib")
 
-require 'erb_lint/cli'
+require "erb_lint/cli"
 
 cli = ERBLint::CLI.new
 exit(cli.run)

data/lib/erb_lint.rb

@@ -1,3 +1,3 @@
 # frozen_string_literal: true
 
-require 'erb_lint/version'
+require "erb_lint/version"

data/lib/erb_lint/all.rb

@@ -1,26 +1,26 @@
 # frozen_string_literal: true
 
-require 'rubocop'
+require "rubocop"
 
-require 'erb_lint'
-require 'erb_lint/corrector'
-require 'erb_lint/file_loader'
-require 'erb_lint/linter_config'
-require 'erb_lint/linter_registry'
-require 'erb_lint/linter'
-require 'erb_lint/offense'
-require 'erb_lint/processed_source'
-require 'erb_lint/runner_config'
-require 'erb_lint/runner'
-require 'erb_lint/stats'
-require 'erb_lint/reporter'
+require "erb_lint"
+require "erb_lint/corrector"
+require "erb_lint/file_loader"
+require "erb_lint/linter_config"
+require "erb_lint/linter_registry"
+require "erb_lint/linter"
+require "erb_lint/offense"
+require "erb_lint/processed_source"
+require "erb_lint/runner_config"
+require "erb_lint/runner"
+require "erb_lint/stats"
+require "erb_lint/reporter"
 
 # Load linters
-Dir[File.expand_path('linters/**/*.rb', __dir__)].each do |file|
+Dir[File.expand_path("linters/**/*.rb", __dir__)].each do |file|
   require file
 end
 
 # Load reporters
-Dir[File.expand_path('reporters/**/*.rb', __dir__)].each do |file|
+Dir[File.expand_path("reporters/**/*.rb", __dir__)].each do |file|
   require file
 end

data/lib/erb_lint/cli.rb

@@ -1,19 +1,19 @@
 # frozen_string_literal: true
 
-require 'erb_lint/all'
-require 'active_support'
-require 'active_support/inflector'
-require 'optparse'
-require 'psych'
-require 'yaml'
-require 'rainbow'
-require 'erb_lint/utils/severity_levels'
+require "erb_lint/all"
+require "active_support"
+require "active_support/inflector"
+require "optparse"
+require "psych"
+require "yaml"
+require "rainbow"
+require "erb_lint/utils/severity_levels"
 
 module ERBLint
   class CLI
     include Utils::SeverityLevels
 
-    DEFAULT_CONFIG_FILENAME = '.erb-lint.yml'
+    DEFAULT_CONFIG_FILENAME = ".erb-lint.yml"
     DEFAULT_LINT_ALL_GLOB = "**/*.html{+*,}.erb"
 
     class ExitWithFailure < RuntimeError; end
@@ -43,7 +43,7 @@ module ERBLint
       ensure_files_exist(lint_files)
 
       if enabled_linter_classes.empty?
-        failure!('no linter available with current configuration')
+        failure!("no linter available with current configuration")
       end
 
       @options[:format] ||= :multiline
@@ -145,7 +145,7 @@ module ERBLint
 
     def correct(processed_source, offenses)
       corrector = ERBLint::Corrector.new(processed_source, offenses)
-      failure!(corrector.diagnostics.join(', ')) if corrector.diagnostics.any?
+      failure!(corrector.diagnostics.join(", ")) if corrector.diagnostics.any?
       corrector
     end
 
@@ -183,7 +183,7 @@ module ERBLint
         else
           @files
             .map { |f| Dir.exist?(f) ? Dir[File.join(f, glob)] : f }
-            .map { |f| f.include?('*') ? Dir[f] : f }
+            .map { |f| f.include?("*") ? Dir[f] : f }
             .flatten
             .map { |f| File.expand_path(f, Dir.pwd) }
             .select { |filename| !excluded?(filename) }
@@ -240,14 +240,14 @@ module ERBLint
     end
 
     def relative_filename(filename)
-      filename.sub("#{File.expand_path('.', Dir.pwd)}/", '')
+      filename.sub("#{File.expand_path(".", Dir.pwd)}/", "")
     end
 
     def runner_config_override
       RunnerConfig.new(
         linters: {}.tap do |linters|
           ERBLint::LinterRegistry.linters.map do |klass|
-            linters[klass.simple_name] = { 'enabled' => enabled_linter_classes.include?(klass) }
+            linters[klass.simple_name] = { "enabled" => enabled_linter_classes.include?(klass) }
           end
         end
       )
@@ -283,10 +283,10 @@ module ERBLint
         end
 
         opts.on("--enable-linters LINTER[,LINTER,...]", Array,
-          "Only use specified linter", "Known linters are: #{known_linter_names.join(', ')}") do |linters|
+          "Only use specified linter", "Known linters are: #{known_linter_names.join(", ")}") do |linters|
           linters.each do |linter|
             unless known_linter_names.include?(linter)
-              failure!("#{linter}: not a valid linter name (#{known_linter_names.join(', ')})")
+              failure!("#{linter}: not a valid linter name (#{known_linter_names.join(", ")})")
             end
           end
           @options[:enabled_linters] = linters
@@ -296,7 +296,7 @@ module ERBLint
           parsed_severity = SEVERITY_CODE_TABLE[level.upcase.to_sym] || (SEVERITY_NAMES & [level.downcase]).first
 
           if parsed_severity.nil?
-            failure!("#{level}: not a valid failure level (#{SEVERITY_NAMES.join(', ')})")
+            failure!("#{level}: not a valid failure level (#{SEVERITY_NAMES.join(", ")})")
           end
           @options[:fail_level] = severity_level_for_name(parsed_severity)
         end
@@ -325,7 +325,7 @@ module ERBLint
 
     def format_options_help
       "Report offenses in the given format: "\
-      "(#{Reporter.available_formats.join(', ')}) (default: multiline)"
+      "(#{Reporter.available_formats.join(", ")}) (default: multiline)"
     end
 
     def invalid_format_error_message(given_format)

data/lib/erb_lint/corrector.rb

@@ -21,7 +21,7 @@ module ERBLint
     end
 
     if ::RuboCop::Version::STRING.to_f >= 0.87
-      require 'rubocop/cop/legacy/corrector'
+      require "rubocop/cop/legacy/corrector"
       BASE = ::RuboCop::Cop::Legacy::Corrector
 
       def diagnostics

data/lib/erb_lint/linter.rb

@@ -15,9 +15,9 @@ module ERBLint
       # `ERBLint::Linters::Compass::Bar.simple_name` #=> "Compass::Bar"
       def inherited(linter)
         super
-        linter.simple_name = if linter.name.start_with?('ERBLint::Linters::')
-          name_parts = linter.name.split('::')
-          name_parts[2..-1].join('::')
+        linter.simple_name = if linter.name.start_with?("ERBLint::Linters::")
+          name_parts = linter.name.split("::")
+          name_parts[2..-1].join("::")
         else
           linter.name
         end

data/lib/erb_lint/linter_config.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'active_support'
-require 'smart_properties'
+require "active_support"
+require "smart_properties"
 
 module ERBLint
   class LinterConfig
@@ -27,7 +27,7 @@ module ERBLint
       allowed_keys = self.class.properties.keys.map(&:to_s)
       given_keys = config.keys
       if (extra_keys = given_keys - allowed_keys).any?
-        raise Error, "Given key is not allowed: #{extra_keys.join(', ')}"
+        raise Error, "Given key is not allowed: #{extra_keys.join(", ")}"
       end
       super(config)
     rescue SmartProperties::InitializationError => e

data/lib/erb_lint/linter_registry.rb

@@ -3,7 +3,7 @@
 module ERBLint
   # Stores all linters available to the application.
   module LinterRegistry
-    CUSTOM_LINTERS_DIR = '.erb-linters'
+    CUSTOM_LINTERS_DIR = ".erb-linters"
     @loaded_linters = []
 
     class << self
@@ -27,7 +27,7 @@ module ERBLint
       end
 
       def load_custom_linters(directory = CUSTOM_LINTERS_DIR)
-        ruby_files = Dir.glob(File.expand_path(File.join(directory, '**', '*.rb')))
+        ruby_files = Dir.glob(File.expand_path(File.join(directory, "**", "*.rb")))
         ruby_files.each { |file| require file }
       end
     end

data/lib/erb_lint/linters/allowed_script_type.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'better_html'
-require 'better_html/tree/tag'
+require "better_html"
+require "better_html/tree/tag"
 
 module ERBLint
   module Linters
@@ -13,7 +13,7 @@ module ERBLint
 
       class ConfigSchema < LinterConfig
         property :allowed_types, accepts: array_of?(String),
-          default: -> { ['text/javascript'] }
+          default: -> { ["text/javascript"] }
         property :allow_blank, accepts: [true, false], default: true, reader: :allow_blank?
         property :disallow_inline_scripts, accepts: [true, false], default: false, reader: :disallow_inline_scripts?
       end
@@ -24,7 +24,7 @@ module ERBLint
         parser.nodes_with_type(:tag).each do |tag_node|
           tag = BetterHtml::Tree::Tag.from_node(tag_node)
           next if tag.closing?
-          next unless tag.name == 'script'
+          next unless tag.name == "script"
 
           if @config.disallow_inline_scripts?
             name_node = tag_node.to_a[1]
@@ -36,7 +36,7 @@ module ERBLint
             next
           end
 
-          type_attribute = tag.attributes['type']
+          type_attribute = tag.attributes["type"]
           type_present = type_attribute.present? && type_attribute.value_node.present?
 
           if !type_present && !@config.allow_blank?
@@ -50,8 +50,8 @@ module ERBLint
             add_offense(
               type_attribute.loc,
               "Avoid using #{type_attribute.value.inspect} as type for `<script>` tag. "\
-              "Must be one of: #{@config.allowed_types.join(', ')}"\
-              "#{' (or no type attribute)' if @config.allow_blank?}."
+              "Must be one of: #{@config.allowed_types.join(", ")}"\
+              "#{" (or no type attribute)" if @config.allow_blank?}."
             )
           end
         end

data/lib/erb_lint/linters/closing_erb_tag_indent.rb

@@ -25,7 +25,7 @@ module ERBLint
             add_offense(
               code_node.loc.end.adjust(begin_pos: -end_spaces.size),
               "Remove newline before `%>` to match start of tag.",
-              ' '
+              " "
             )
           elsif start_with_newline && !end_with_newline
             add_offense(
@@ -39,7 +39,7 @@ module ERBLint
               add_offense(
                 code_node.loc.end.adjust(begin_pos: -current_indent.size),
                 "Indent `%>` on column #{erb_node.loc.column} to match start of tag.",
-                ' ' * erb_node.loc.column
+                " " * erb_node.loc.column
               )
             end
           end

data/lib/erb_lint/linters/deprecated_classes.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'better_html'
-require 'better_html/parser'
+require "better_html"
+require "better_html/parser"
 
 module ERBLint
   module Linters
@@ -11,7 +11,7 @@ module ERBLint
 
       class RuleSet
         include SmartProperties
-        property :suggestion, accepts: String, default: ''
+        property :suggestion, accepts: String, default: ""
         property :deprecated, accepts: LinterConfig.array_of?(String), default: -> { [] }
       end
 
@@ -57,9 +57,9 @@ module ERBLint
       def class_name_with_loc(processed_source)
         Enumerator.new do |yielder|
           tags(processed_source).each do |tag|
-            class_value = tag.attributes['class']&.value
+            class_value = tag.attributes["class"]&.value
             next unless class_value
-            class_value.split(' ').each do |class_name|
+            class_value.split(" ").each do |class_name|
               yielder.yield(class_name, tag.loc)
             end
           end
@@ -69,7 +69,7 @@ module ERBLint
       def text_tags_content(processed_source)
         Enumerator.new do |yielder|
           script_tags(processed_source)
-            .select { |tag| tag.attributes['type']&.value == 'text/html' }
+            .select { |tag| tag.attributes["type"]&.value == "text/html" }
             .each do |tag|
               index = processed_source.ast.to_a.find_index(tag.node)
               next_node = processed_source.ast.to_a[index + 1]
@@ -80,7 +80,7 @@ module ERBLint
       end
 
       def script_tags(processed_source)
-        tags(processed_source).select { |tag| tag.name == 'script' }
+        tags(processed_source).select { |tag| tag.name == "script" }
       end
 
       def tags(processed_source)

data/lib/erb_lint/linters/erb_safety.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'better_html'
-require 'better_html/test_helper/safe_erb_tester'
+require "better_html"
+require "better_html/test_helper/safe_erb_tester"
 
 module ERBLint
   module Linters

data/lib/erb_lint/linters/extra_newline.rb

@@ -22,7 +22,7 @@ module ERBLint
 
       def autocorrect(_processed_source, offense)
         lambda do |corrector|
-          corrector.replace(offense.source_range, '')
+          corrector.replace(offense.source_range, "")
         end
       end
     end

data/lib/erb_lint/linters/final_newline.rb

@@ -28,7 +28,7 @@ module ERBLint
           if final_newline.empty?
             add_offense(
               processed_source.to_source_range(file_content.size...file_content.size),
-              'Missing a trailing newline at the end of the file.',
+              "Missing a trailing newline at the end of the file.",
               :insert
             )
           else
@@ -36,7 +36,7 @@ module ERBLint
               processed_source.to_source_range(
                 (file_content.size - final_newline.size + 1)...file_content.size
               ),
-              'Remove multiple trailing newline at the end of the file.',
+              "Remove multiple trailing newline at the end of the file.",
               :remove
             )
           end

data/lib/erb_lint/linters/hard_coded_string.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 require "set"
-require 'better_html/tree/tag'
-require 'active_support/core_ext/string/inflections'
+require "better_html/tree/tag"
+require "active_support/core_ext/string/inflections"
 
 module ERBLint
   module Linters
@@ -13,37 +13,37 @@ module ERBLint
       MissingCorrector = Class.new(StandardError)
       MissingI18nLoadPath = Class.new(StandardError)
 
-      ALLOWED_CORRECTORS = %w(
-        I18nCorrector
-        RuboCop::Corrector::I18n::HardCodedString
-      )
-
-      NON_TEXT_TAGS = Set.new(%w(script style xmp iframe noembed noframes listing))
-      BLACK_LISTED_TEXT = Set.new(%w(
-         
-        &
-        <
-        >
-        "
-        ©
-        ®
-        ™
-        …
-        —
-        •
-        “
-        ”
-        ‘
-        ’
-        ←
-        →
-        ↓
-        ↑
-      ))
+      ALLOWED_CORRECTORS = ["I18nCorrector", "RuboCop::Corrector::I18n::HardCodedString"]
+
+      NON_TEXT_TAGS = Set.new(["script", "style", "xmp", "iframe", "noembed", "noframes", "listing"])
+      TEXT_NOT_ALLOWED = Set.new([
+        " ",
+        "&",
+        "<",
+        ">",
+        """,
+        "©",
+        "®",
+        "™",
+        "…",
+        "—",
+        "•",
+        "“",
+        "”",
+        "‘",
+        "’",
+        "←",
+        "→",
+        "↓",
+        "↑",
+        " ",
+        " ",
+        " ",
+      ])
 
       class ConfigSchema < LinterConfig
         property :corrector, accepts: Hash, required: false, default: -> { {} }
-        property :i18n_load_path, accepts: String, required: false, default: ''
+        property :i18n_load_path, accepts: String, required: false, default: ""
       end
       self.config_schema = ConfigSchema
 
@@ -85,7 +85,7 @@ module ERBLint
         return unless string.strip.length > 1
         node = ::RuboCop::AST::StrNode.new(:str, [string])
         corrector = klass.new(node, processed_source.filename, corrector_i18n_load_path, offense.source_range)
-        corrector.autocorrect(tag_start: '<%= ', tag_end: ' %>')
+        corrector.autocorrect(tag_start: "<%= ", tag_end: " %>")
       rescue MissingCorrector, MissingI18nLoadPath
         nil
       end
@@ -93,20 +93,20 @@ module ERBLint
       private
 
       def check_string?(str)
-        string = str.gsub(/\s*/, '')
-        string.length > 1 && !BLACK_LISTED_TEXT.include?(string)
+        string = str.gsub(/\s*/, "")
+        string.length > 1 && !TEXT_NOT_ALLOWED.include?(string)
       end
 
       def load_corrector
-        corrector_name = @config['corrector'].fetch('name') { raise MissingCorrector }
+        corrector_name = @config["corrector"].fetch("name") { raise MissingCorrector }
         raise ForbiddenCorrector unless ALLOWED_CORRECTORS.include?(corrector_name)
-        require @config['corrector'].fetch('path') { raise MissingCorrector }
+        require @config["corrector"].fetch("path") { raise MissingCorrector }
 
         corrector_name.safe_constantize
       end
 
       def corrector_i18n_load_path
-        @config['corrector'].fetch('i18n_load_path') { raise MissingI18nLoadPath }
+        @config["corrector"].fetch("i18n_load_path") { raise MissingI18nLoadPath }
       end
 
       def non_text_tag?(processed_source, text_node)

data/lib/erb_lint/linters/no_javascript_tag_helper.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require 'better_html'
-require 'better_html/ast/node'
-require 'better_html/test_helper/ruby_node'
-require 'erb_lint/utils/block_map'
-require 'erb_lint/utils/ruby_to_erb'
+require "better_html"
+require "better_html/ast/node"
+require "better_html/test_helper/ruby_node"
+require "erb_lint/utils/block_map"
+require "erb_lint/utils/ruby_to_erb"
 
 module ERBLint
   module Linters
@@ -21,7 +21,7 @@ module ERBLint
         parser.ast.descendants(:erb).each do |erb_node|
           indicator_node, _, code_node, _ = *erb_node
           indicator = indicator_node&.loc&.source
-          next if indicator == '#'
+          next if indicator == "#"
           source = code_node.loc.source
 
           ruby_node =
@@ -63,10 +63,10 @@ module ERBLint
         return unless (0..2).cover?(argument_nodes.size)
 
         script_content = unless argument_nodes.first&.type?(:hash)
-          Utils::RubyToERB.ruby_to_erb(argument_nodes.first, '==')
+          Utils::RubyToERB.ruby_to_erb(argument_nodes.first, "==")
         end
         arguments = if argument_nodes.last&.type?(:hash)
-          ' ' + Utils::RubyToERB.html_options_to_tag_attributes(argument_nodes.last)
+          " " + Utils::RubyToERB.html_options_to_tag_attributes(argument_nodes.last)
         end
 
         return if end_node && script_content

data/lib/erb_lint/linters/require_input_autocomplete.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'better_html'
-require 'better_html/tree/tag'
+require "better_html"
+require "better_html/tree/tag"
 
 module ERBLint
   module Linters
@@ -13,7 +13,6 @@ module ERBLint
         "date",
         "datetime-local",
         "email",
-        "hidden",
         "month",
         "number",
         "password",
@@ -33,7 +32,6 @@ module ERBLint
         :text_field_tag,
         :utf8_enforcer_tag,
         :month_field_tag,
-        :hidden_field_tag,
         :number_field_tag,
         :password_field_tag,
         :search_field_tag,
@@ -56,8 +54,8 @@ module ERBLint
         parser.nodes_with_type(:tag).each do |tag_node|
           tag = BetterHtml::Tree::Tag.from_node(tag_node)
 
-          autocomplete_attribute = tag.attributes['autocomplete']
-          type_attribute = tag.attributes['type']
+          autocomplete_attribute = tag.attributes["autocomplete"]
+          type_attribute = tag.attributes["type"]
 
           next if !html_input_tag?(tag) || autocomplete_present?(autocomplete_attribute)
           next unless html_type_requires_autocomplete_attribute?(type_attribute)
@@ -76,7 +74,7 @@ module ERBLint
       end
 
       def html_input_tag?(tag)
-        !tag.closing? && tag.name == 'input'
+        !tag.closing? && tag.name == "input"
       end
 
       def html_type_requires_autocomplete_attribute?(type_attribute)
@@ -110,7 +108,7 @@ module ERBLint
       end
 
       def code_comment?(indicator_node)
-        indicator_node&.loc&.source == '#'
+        indicator_node&.loc&.source == "#"
       end
 
       def extract_ruby_node(source)

data/lib/erb_lint/linters/require_script_nonce.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'better_html'
-require 'better_html/tree/tag'
+require "better_html"
+require "better_html/tree/tag"
 
 module ERBLint
   module Linters
@@ -22,7 +22,7 @@ module ERBLint
       def find_html_script_tags(parser)
         parser.nodes_with_type(:tag).each do |tag_node|
           tag = BetterHtml::Tree::Tag.from_node(tag_node)
-          nonce_attribute = tag.attributes['nonce']
+          nonce_attribute = tag.attributes["nonce"]
 
           next if !html_javascript_tag?(tag) || nonce_present?(nonce_attribute)
 
@@ -40,16 +40,16 @@ module ERBLint
 
       def html_javascript_tag?(tag)
         !tag.closing? &&
-          (tag.name == 'script' && !html_javascript_type_attribute?(tag))
+          (tag.name == "script" && !html_javascript_type_attribute?(tag))
       end
 
       def html_javascript_type_attribute?(tag)
-        type_attribute = tag.attributes['type']
+        type_attribute = tag.attributes["type"]
 
         type_attribute &&
           type_attribute.value_node.present? &&
-          type_attribute.value_node.to_a[1] != 'text/javascript' &&
-          type_attribute.value_node.to_a[1] != 'application/javascript'
+          type_attribute.value_node.to_a[1] != "text/javascript" &&
+          type_attribute.value_node.to_a[1] != "application/javascript"
       end
 
       def find_rails_helper_script_tags(parser)
@@ -79,7 +79,7 @@ module ERBLint
       end
 
       def code_comment?(indicator_node)
-        indicator_node&.loc&.source == '#'
+        indicator_node&.loc&.source == "#"
       end
 
       def extract_ruby_node(source)

data/lib/erb_lint/linters/right_trim.rb

@@ -8,7 +8,7 @@ module ERBLint
       include LinterRegistry
 
       class ConfigSchema < LinterConfig
-        property :enforced_style, accepts: ['-', '='], default: '-'
+        property :enforced_style, accepts: ["-", "="], default: "-"
       end
       self.config_schema = ConfigSchema
 

data/lib/erb_lint/linters/rubocop.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require 'better_html'
-require 'tempfile'
-require 'erb_lint/utils/offset_corrector'
+require "better_html"
+require "tempfile"
+require "erb_lint/utils/offset_corrector"
 
 module ERBLint
   module Linters
@@ -25,7 +25,7 @@ module ERBLint
         super
         @only_cops = @config.only
         custom_config = config_from_hash(@config.rubocop_config)
-        @rubocop_config = ::RuboCop::ConfigLoader.merge_with_default(custom_config, '')
+        @rubocop_config = ::RuboCop::ConfigLoader.merge_with_default(custom_config, "")
       end
 
       def run(processed_source)
@@ -68,13 +68,13 @@ module ERBLint
 
       def inspect_content(processed_source, erb_node)
         indicator, _, code_node, = *erb_node
-        return if indicator&.children&.first == '#'
+        return if indicator&.children&.first == "#"
 
         original_source = code_node.loc.source
-        trimmed_source = original_source.sub(BLOCK_EXPR, '').sub(SUFFIX_EXPR, '')
+        trimmed_source = original_source.sub(BLOCK_EXPR, "").sub(SUFFIX_EXPR, "")
         alignment_column = code_node.loc.column
         offset = code_node.loc.begin_pos - alignment_column
-        aligned_source = "#{' ' * alignment_column}#{trimmed_source}"
+        aligned_source = "#{" " * alignment_column}#{trimmed_source}"
 
         source = rubocop_processed_source(aligned_source, processed_source.filename)
         return unless source.valid_syntax?
@@ -156,10 +156,10 @@ module ERBLint
       end
 
       def config_from_hash(hash)
-        inherit_from = hash&.delete('inherit_from')
+        inherit_from = hash&.delete("inherit_from")
         resolve_inheritance(hash, inherit_from)
 
-        tempfile_from('.erblint-rubocop', hash.to_yaml) do |tempfile|
+        tempfile_from(".erblint-rubocop", hash.to_yaml) do |tempfile|
           ::RuboCop::ConfigLoader.load_file(tempfile.path)
         end
       end
@@ -174,7 +174,7 @@ module ERBLint
       end
 
       def base_configs(inherit_from)
-        regex = URI::DEFAULT_PARSER.make_regexp(%w(http https))
+        regex = URI::DEFAULT_PARSER.make_regexp(["http", "https"])
         configs = Array(inherit_from).compact.map do |base_name|
           if base_name =~ /\A#{regex}\z/
             ::RuboCop::ConfigLoader.load_file(::RuboCop::RemoteConfig.new(base_name, Dir.pwd))

data/lib/erb_lint/linters/rubocop_text.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'rubocop'
+require_relative "rubocop"
 
 module ERBLint
   module Linters

data/lib/erb_lint/linters/self_closing_tag.rb

@@ -11,10 +11,8 @@ module ERBLint
       end
       self.config_schema = ConfigSchema
 
-      SELF_CLOSING_TAGS = %w(
-        area base br col command embed hr input keygen
-        link menuitem meta param source track wbr img
-      )
+      SELF_CLOSING_TAGS = ["area", "base", "br", "col", "command", "embed", "hr", "input", "keygen", "link",
+                           "menuitem", "meta", "param", "source", "track", "wbr", "img",]
 
       def run(processed_source)
         processed_source.ast.descendants(:tag).each do |tag_node|
@@ -26,7 +24,7 @@ module ERBLint
             add_offense(
               start_solidus.loc,
               "Tag `#{tag.name}` is a void element, it must not start with `</`.",
-              ''
+              ""
             )
           end
 
@@ -34,7 +32,7 @@ module ERBLint
             add_offense(
               tag_node.loc.end.offset(-1),
               "Tag `#{tag.name}` is self-closing, it must end with `/>`.",
-              '/'
+              "/"
             )
           end
 
@@ -43,7 +41,7 @@ module ERBLint
           add_offense(
             end_solidus.loc,
             "Tag `#{tag.name}` is a void element, it must end with `>` and not `/>`.",
-            ''
+            ""
           )
         end
       end

data/lib/erb_lint/linters/space_around_erb_tag.rb

@@ -15,7 +15,7 @@ module ERBLint
         processed_source.ast.descendants(:erb).each do |erb_node|
           indicator_node, ltrim, code_node, rtrim = *erb_node
           indicator = indicator_node&.loc&.source
-          next if indicator == '#' || indicator == '%'
+          next if indicator == "#" || indicator == "%"
           code = code_node.children.first
 
           start_spaces = code.match(START_SPACES)&.captures&.first || ""
@@ -23,8 +23,8 @@ module ERBLint
             add_offense(
               code_node.loc.resize(start_spaces.size),
               "Use 1 space after `<%#{indicator}#{ltrim&.loc&.source}` "\
-              "instead of #{start_spaces.size} space#{'s' if start_spaces.size > 1}.",
-              ' '
+              "instead of #{start_spaces.size} space#{"s" if start_spaces.size > 1}.",
+              " "
             )
           elsif start_spaces.count("\n") > 1
             lines = start_spaces.split("\n", -1)
@@ -41,8 +41,8 @@ module ERBLint
             add_offense(
               code_node.loc.end.adjust(begin_pos: -end_spaces.size),
               "Use 1 space before `#{rtrim&.loc&.source}%>` "\
-              "instead of #{end_spaces.size} space#{'s' if start_spaces.size > 1}.",
-              ' '
+              "instead of #{end_spaces.size} space#{"s" if start_spaces.size > 1}.",
+              " "
             )
           elsif end_spaces.count("\n") > 1
             lines = end_spaces.split("\n", -1)

data/lib/erb_lint/linters/space_in_html_tag.rb

@@ -50,7 +50,7 @@ module ERBLint
         add_offense(
           processed_source.to_source_range(range),
           "Extra space detected where there should be no space.",
-          ''
+          ""
         )
       end
 
@@ -60,24 +60,24 @@ module ERBLint
 
       def single_space(processed_source, range, accept_newline: false)
         chars = processed_source.file_content[range]
-        return if chars == ' '
+        return if chars == " "
 
         newlines = chars.include?("\n")
-        expected = newlines && accept_newline ? "\n#{chars.split("\n", -1).last}" : ' '
+        expected = newlines && accept_newline ? "\n#{chars.split("\n", -1).last}" : " "
         non_space = chars.match(/([^[[:space:]]])/m)
 
         if non_space && !non_space.captures.empty?
           add_offense(
             processed_source.to_source_range(range),
             "Non-whitespace character(s) detected: "\
-              "#{non_space.captures.map(&:inspect).join(', ')}.",
+              "#{non_space.captures.map(&:inspect).join(", ")}.",
             expected
           )
         elsif newlines && accept_newline
           if expected != chars
             add_offense(
               processed_source.to_source_range(range),
-              "#{chars.empty? ? 'No' : 'Extra'} space detected where there should be "\
+              "#{chars.empty? ? "No" : "Extra"} space detected where there should be "\
                 "a single space or a single line break.",
               expected
             )
@@ -85,7 +85,7 @@ module ERBLint
         else
           add_offense(
             processed_source.to_source_range(range),
-            "#{chars.empty? ? 'No' : 'Extra'} space detected where there should be a single space.",
+            "#{chars.empty? ? "No" : "Extra"} space detected where there should be a single space.",
             expected
           )
         end

data/lib/erb_lint/linters/space_indentation.rb

@@ -23,7 +23,7 @@ module ERBLint
             add_offense(
               processed_source.to_source_range(document_pos...(document_pos + spaces.length)),
               "Indent with spaces instead of tabs.",
-              spaces.gsub("\t", ' ' * @config.tab_width)
+              spaces.gsub("\t", " " * @config.tab_width)
             )
           end
 

data/lib/erb_lint/linters/trailing_whitespace.rb

@@ -25,7 +25,7 @@ module ERBLint
 
       def autocorrect(_processed_source, offense)
         lambda do |corrector|
-          corrector.replace(offense.source_range, '')
+          corrector.replace(offense.source_range, "")
         end
       end
     end

data/lib/erb_lint/reporter.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
-require 'active_support/core_ext/class'
-require 'active_support/core_ext/module/delegation'
+require "active_support/core_ext/class"
+require "active_support/core_ext/module/delegation"
 
 module ERBLint
   class Reporter

data/lib/erb_lint/reporters/compact_reporter.rb

@@ -5,7 +5,7 @@ module ERBLint
     class CompactReporter < Reporter
       def preview
         puts "Linting #{stats.files} files with "\
-          "#{stats.linters} #{'autocorrectable ' if autocorrect}linters..."
+          "#{stats.linters} #{"autocorrectable " if autocorrect}linters..."
       end
 
       def show

data/lib/erb_lint/reporters/multiline_reporter.rb

@@ -9,7 +9,7 @@ module ERBLint
       def format_offense(filename, offense)
         <<~EOF
 
-          #{offense.message}#{Rainbow(' (not autocorrected)').red if autocorrect}
+          #{offense.message}#{Rainbow(" (not autocorrected)").red if autocorrect}
           In file: #{filename}:#{offense.line_number}
         EOF
       end

data/lib/erb_lint/runner.rb

@@ -8,7 +8,7 @@ module ERBLint
     def initialize(file_loader, config)
       @file_loader = file_loader
       @config = config || RunnerConfig.default
-      raise ArgumentError, 'expect `config` to be a RunnerConfig instance' unless @config.is_a?(RunnerConfig)
+      raise ArgumentError, "expect `config` to be a RunnerConfig instance" unless @config.is_a?(RunnerConfig)
 
       linter_classes = LinterRegistry.linters.select { |klass| @config.for_linter(klass).enabled? }
       @linters = linter_classes.map do |linter_class|

data/lib/erb_lint/runner_config.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'erb_lint/runner_config_resolver'
+require "erb_lint/runner_config_resolver"
 
 module ERBLint
   class RunnerConfig
@@ -9,7 +9,7 @@ module ERBLint
     def initialize(config = nil, file_loader = nil)
       @config = (config || {}).dup.deep_stringify_keys
 
-      resolver.resolve_inheritance_from_gems(@config, @config.delete('inherit_gem'))
+      resolver.resolve_inheritance_from_gems(@config, @config.delete("inherit_gem"))
       resolver.resolve_inheritance(@config, file_loader) if file_loader
       @config.delete("inherit_from")
     end
@@ -24,7 +24,7 @@ module ERBLint
       elsif klass.is_a?(Class) && klass <= ERBLint::Linter
         klass.simple_name
       else
-        raise ArgumentError, 'expected String or linter class'
+        raise ArgumentError, "expected String or linter class"
       end
       linter_klass = LinterRegistry.find_by_name(klass_name)
       raise Error, "#{klass_name}: linter not found (is it loaded?)" unless linter_klass
@@ -32,7 +32,7 @@ module ERBLint
     end
 
     def global_exclude
-      @config['exclude'] || []
+      @config["exclude"] || []
     end
 
     def merge(other_config)
@@ -75,13 +75,13 @@ module ERBLint
     private
 
     def linters_config
-      @config['linters'] || {}
+      @config["linters"] || {}
     end
 
     def config_hash_for_linter(klass_name)
       config_hash = linters_config[klass_name] || {}
-      config_hash['exclude'] ||= []
-      config_hash['exclude'].concat(global_exclude) if config_hash['exclude'].is_a?(Array)
+      config_hash["exclude"] ||= []
+      config_hash["exclude"].concat(global_exclude) if config_hash["exclude"].is_a?(Array)
       config_hash
     end
 

data/lib/erb_lint/runner_config_resolver.rb

@@ -24,7 +24,7 @@
 module ERBLint
   class RunnerConfigResolver
     def resolve_inheritance(hash, file_loader)
-      inherited_files = Array(hash['inherit_from'])
+      inherited_files = Array(hash["inherit_from"])
       base_configs(file_loader, inherited_files).reverse_each do |base_config|
         base_config.each do |k, v|
           next unless v.is_a?(Hash)
@@ -36,12 +36,12 @@ module ERBLint
 
     def resolve_inheritance_from_gems(hash, gems)
       (gems || {}).each_pair do |gem_name, config_path|
-        raise(ArgumentError, "can't inherit configuration from the erb-lint gem") if gem_name == 'erb-lint'
+        raise(ArgumentError, "can't inherit configuration from the erb-lint gem") if gem_name == "erb-lint"
 
-        hash['inherit_from'] = Array(hash['inherit_from'])
+        hash["inherit_from"] = Array(hash["inherit_from"])
         Array(config_path).reverse_each do |path|
           # Put gem configuration first so local configuration overrides it.
-          hash['inherit_from'].unshift(gem_config_path(gem_name, path))
+          hash["inherit_from"].unshift(gem_config_path(gem_name, path))
         end
       end
     end

data/lib/erb_lint/stats.rb

@@ -2,12 +2,12 @@
 module ERBLint
   class Stats
     attr_accessor :ignored,
-                  :found,
-                  :corrected,
-                  :exceptions,
-                  :linters,
-                  :files,
-                  :processed_files
+      :found,
+      :corrected,
+      :exceptions,
+      :linters,
+      :files,
+      :processed_files
 
     def initialize(
       ignored: 0,

data/lib/erb_lint/utils/block_map.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'better_html/ast/node'
-require 'better_html/test_helper/ruby_node'
+require "better_html/ast/node"
+require "better_html/test_helper/ruby_node"
 
 module ERBLint
   module Utils

data/lib/erb_lint/utils/offset_corrector.rb

@@ -63,7 +63,7 @@ module ERBLint
           node_or_range
         else
           raise TypeError,
-            'Expected a Parser::Source::Range, Comment or ' \
+            "Expected a Parser::Source::Range, Comment or " \
                         "Rubocop::AST::Node, got #{node_or_range.class}"
         end
       end

data/lib/erb_lint/utils/ruby_to_erb.rb

@@ -9,10 +9,10 @@ module ERBLint
         def html_options_to_tag_attributes(hash_node)
           hash_node.children.map do |pair_node|
             key_node, value_node = *pair_node
-            key = ruby_to_erb(key_node, '=') { |s| s.tr('_', '-') }
-            value = ruby_to_erb(value_node, '=') { |s| escape_quote(s) }
-            [key, "\"#{value}\""].join('=')
-          end.join(' ')
+            key = ruby_to_erb(key_node, "=") { |s| s.tr("_", "-") }
+            value = ruby_to_erb(value_node, "=") { |s| escape_quote(s) }
+            [key, "\"#{value}\""].join("=")
+          end.join(" ")
         end
 
         def ruby_to_erb(node, indicator = nil, &block)
@@ -43,7 +43,7 @@ module ERBLint
         end
 
         def escape_quote(str)
-          str.gsub('"', '"')
+          str.gsub('"', """)
         end
       end
     end

data/lib/erb_lint/utils/severity_levels.rb

@@ -3,10 +3,10 @@
 module ERBLint
   module Utils
     module SeverityLevels
-      SEVERITY_NAMES = %i[info refactor convention warning error fatal].freeze
+      SEVERITY_NAMES = [:info, :refactor, :convention, :warning, :error, :fatal].freeze
 
       SEVERITY_CODE_TABLE = { I: :info, R: :refactor, C: :convention,
-                              W: :warning, E: :error, F: :fatal }.freeze
+                              W: :warning, E: :error, F: :fatal, }.freeze
 
       def severity_level_for_name(name)
         SEVERITY_NAMES.index(name || :error) + 1

data/lib/erb_lint/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ERBLint
-  VERSION = '0.1.0'
+  VERSION = "0.1.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: erb_lint
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Justin Chan
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-07-26 00:00:00.000000000 Z
+date: 2021-07-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: better_html