html-proofer 3.13.0 → 3.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f4a7b3e2288d095d98ef82cd84a0e4f5b05d2b2ef5ade6b14bb8b33d469b75dd
-  data.tar.gz: bf6735289b6a63638a9b5d3d78491191257463818a8ff99eaef4df3edab1dbbe
+  metadata.gz: cf06ea39b2c106240c1d6c10261b50bab110fa5620451690970c8226bd0022cb
+  data.tar.gz: 70446e5bf8162760c6583f53917c3774ad165d29af0bd01ae600c6069d09360b
 SHA512:
-  metadata.gz: 362197c64a8fc7c59ae57b191d933fd372c9a6821160d2e09f4513748ab26669a1cc0de3b705016bc9ab234c5f4f7491b66f99ae429d52122a0e3aa1b3f31647
-  data.tar.gz: 0d27c7f599be1b375a3862b3b67ec7762fd0aee81dde7ecbae16e5a7eaefc255def973f1f926fe6410054c569a31a8a039dcaad700afa8b07e6ea5bb5f606f93
+  metadata.gz: ab6ef8adc5d80cd409f1cd6248ce7b969f69c219b6878e8d4614852dfc8211430db2fd6863c5e5b03a42326dc315e5538d939f506f95a774b1f48df6a9c95b06
+  data.tar.gz: 603ff3aad5001b0484a753dda3c9b72fa6bac15e94d767f8cb8aaf009810c21b2e7cf5ee039d9261b05c0bb888cad85aa9da769ff0fc13489234bb841c3ee858

data/bin/htmlproofer

@@ -47,6 +47,7 @@ Mercenary.program(:htmlproofer) do |p|
   p.option 'typhoeus_config', '--typhoeus-config CONFIG', String, 'JSON-formatted string of Typhoeus config. Will override the html-proofer defaults.'
   p.option 'url_ignore', '--url-ignore link1,[link2,...]', Array, 'A comma-separated list of Strings or RegExps containing URLs that are safe to ignore. It affects all HTML attributes. Note that non-HTTP(S) URIs are always ignored'
   p.option 'url_swap', '--url-swap re:string,[re:string,...]', Array, 'A comma-separated list containing key-value pairs of `RegExp => String`. It transforms URLs that match `RegExp` into `String` via `gsub`. The escape sequences `\\:` should be used to produce literal `:`s.'
+  p.option 'root_dir', '--root-folder PATH', String, 'The absolute path to the directory serving your html-files. Used when running html-proofer on a file, rather than a directory.'
 
   p.action do |args, opts|
     args = ['.'] if args.empty?
@@ -56,9 +57,7 @@ Mercenary.program(:htmlproofer) do |p|
 
     # prepare everything to go to proofer
     p.options.reject { |o| opts[o.config_key].nil? }.each do |option|
-      if opts[option.config_key].is_a?(Array)
-        opts[option.config_key] = opts[option.config_key].map { |i| HTMLProofer::Configuration.to_regex?(i) }
-      end
+      opts[option.config_key] = opts[option.config_key].map { |i| HTMLProofer::Configuration.to_regex?(i) } if opts[option.config_key].is_a?(Array)
       options[option.config_key.to_sym] = opts[option.config_key]
     end
 
@@ -82,9 +81,7 @@ Mercenary.program(:htmlproofer) do |p|
     options[:validation][:report_missing_names] = opts['report_missing_names'] unless opts['report_missing_names'].nil?
     options[:validation][:report_invalid_tags] = opts['report_invalid_tags'] unless opts['report_invalid_tags'].nil?
 
-    unless opts['typhoeus_config'].nil?
-      options[:typhoeus] = HTMLProofer::Configuration.parse_json_option('typhoeus_config', opts['typhoeus_config'])
-    end
+    options[:typhoeus] = HTMLProofer::Configuration.parse_json_option('typhoeus_config', opts['typhoeus_config']) unless opts['typhoeus_config'].nil?
 
     unless opts['timeframe'].nil?
       options[:cache] ||= {}

data/lib/html-proofer.rb

@@ -17,38 +17,38 @@ require 'fileutils'
 begin
   require 'awesome_print'
   require 'pry-byebug'
-rescue LoadError; end
+rescue LoadError; end # rubocop:disable Lint/HandleExceptions
 module HTMLProofer
-  def check_file(file, options = {})
+  def self.check_file(file, options = {})
     raise ArgumentError unless file.is_a?(String)
     raise ArgumentError, "#{file} does not exist" unless File.exist?(file)
+
     options[:type] = :file
     HTMLProofer::Runner.new(file, options)
   end
-  module_function :check_file
 
-  def check_directory(directory, options = {})
+  def self.check_directory(directory, options = {})
     raise ArgumentError unless directory.is_a?(String)
     raise ArgumentError, "#{directory} does not exist" unless Dir.exist?(directory)
+
     options[:type] = :directory
     HTMLProofer::Runner.new([directory], options)
   end
-  module_function :check_directory
 
-  def check_directories(directories, options = {})
+  def self.check_directories(directories, options = {})
     raise ArgumentError unless directories.is_a?(Array)
+
     options[:type] = :directory
     directories.each do |directory|
       raise ArgumentError, "#{directory} does not exist" unless Dir.exist?(directory)
     end
     HTMLProofer::Runner.new(directories, options)
   end
-  module_function :check_directories
 
-  def check_links(links, options = {})
+  def self.check_links(links, options = {})
     raise ArgumentError unless links.is_a?(Array)
+
     options[:type] = :links
     HTMLProofer::Runner.new(links, options)
   end
-  module_function :check_links
 end

data/lib/html-proofer/cache.rb

@@ -9,7 +9,7 @@ module HTMLProofer
     include HTMLProofer::Utils
 
     DEFAULT_STORAGE_DIR = File.join('tmp', '.htmlproofer')
-    DEFAULT_CACHE_FILE_NAME = 'cache.log'.freeze
+    DEFAULT_CACHE_FILE_NAME = 'cache.log'
 
     attr_reader :exists, :cache_log, :storage_dir, :cache_file
 
@@ -120,9 +120,8 @@ module HTMLProofer
       @cache_log.each_pair do |url, cache|
         if within_timeframe?(cache['time'])
           next if cache['message'].empty? # these were successes to skip
-          urls_to_check[url] = cache['filenames'] # these are failures to retry
         else
-          urls_to_check[url] = cache['filenames'] # pass or fail, recheck expired links
+          urls_to_check[url] = cache['filenames'] # recheck expired links
         end
       end
       urls_to_check
@@ -142,23 +141,16 @@ module HTMLProofer
     end
 
     def setup_cache!(options)
-      @storage_dir = if options[:storage_dir]
-                       options[:storage_dir]
-                     else
-                       DEFAULT_STORAGE_DIR
-                     end
+      @storage_dir = options[:storage_dir] || DEFAULT_STORAGE_DIR
 
       FileUtils.mkdir_p(storage_dir) unless Dir.exist?(storage_dir)
 
-      cache_file_name = if options[:cache_file]
-                          options[:cache_file]
-                        else
-                          DEFAULT_CACHE_FILE_NAME
-                        end
+      cache_file_name = options[:cache_file] || DEFAULT_CACHE_FILE_NAME
 
       @cache_file = File.join(storage_dir, cache_file_name)
 
       return unless File.exist?(cache_file)
+
       contents = File.read(cache_file)
       @cache_log = contents.empty? ? {} : JSON.parse(contents)
     end
@@ -174,7 +166,7 @@ module HTMLProofer
       when :days
         @cache_datetime - measurement
       when :hours
-        @cache_datetime - Rational(measurement/24.0)
+        @cache_datetime - Rational(measurement / 24.0)
       end.to_time
     end
   end

data/lib/html-proofer/check.rb

@@ -29,6 +29,7 @@ module HTMLProofer
 
     def add_to_external_urls(url)
       return if @external_urls[url]
+
       add_path_for_url(url)
     end
 
@@ -45,6 +46,7 @@ module HTMLProofer
 
       ObjectSpace.each_object(Class) do |c|
         next unless c.superclass == self
+
         classes << c
       end
 

data/lib/html-proofer/check/favicon.rb

@@ -6,22 +6,24 @@ class FaviconCheck < ::HTMLProofer::Check
     @html.xpath('//link[not(ancestor::pre or ancestor::code)]').each do |node|
       favicon = create_element(node)
       next if favicon.ignore?
+
       found = true if favicon.rel.split(' ').last.eql? 'icon'
       break if found
     end
 
     return if found
 
-    return if is_immediate_redirect?
+    return if immediate_redirect?
 
     add_issue('no favicon specified')
   end
 
   private
 
-  def is_immediate_redirect?
-    # allow any instant-redirect meta tag
-    @html.xpath("//meta[@http-equiv='refresh']").attribute('content').value.start_with? '0;' rescue false
+  # allow any instant-redirect meta tag
+  def immediate_redirect?
+    @html.xpath("//meta[@http-equiv='refresh']").attribute('content').value.start_with? '0;'
+  rescue StandardError
+    false
   end
-
 end

data/lib/html-proofer/check/html.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
 class HtmlCheck < ::HTMLProofer::Check
-  SCRIPT_EMBEDS_MSG = /Element script embeds close tag/
-  INVALID_TAG_MSG = /Tag ([\w\-:]+) invalid/
-  INVALID_PREFIX = /Namespace prefix/
-  PARSE_ENTITY_REF = /htmlParseEntityRef: no name/
+  SCRIPT_EMBEDS_MSG = /Element script embeds close tag/.freeze
+  INVALID_TAG_MSG = /Tag ([\w\-:]+) invalid/.freeze
+  INVALID_PREFIX = /Namespace prefix/.freeze
+  PARSE_ENTITY_REF = /htmlParseEntityRef: no name/.freeze
 
   def run
     @html.errors.each do |error|

data/lib/html-proofer/check/images.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 class ImageCheck < ::HTMLProofer::Check
-  SCREEN_SHOT_REGEX = /Screen(?: |%20)Shot(?: |%20)\d+-\d+-\d+(?: |%20)at(?: |%20)\d+.\d+.\d+/
+  SCREEN_SHOT_REGEX = /Screen(?: |%20)Shot(?: |%20)\d+-\d+-\d+(?: |%20)at(?: |%20)\d+.\d+.\d+/.freeze
 
   def empty_alt_tag?
     @img.alt.nil? || @img.alt.strip.empty?
@@ -38,13 +38,9 @@ class ImageCheck < ::HTMLProofer::Check
         add_issue("internal image #{@img.url} does not exist", line: line, content: content)
       end
 
-      if empty_alt_tag? && !@img.ignore_empty_alt? && !@img.ignore_alt?
-        add_issue("image #{@img.url} does not have an alt attribute", line: line, content: content)
-      end
+      add_issue("image #{@img.url} does not have an alt attribute", line: line, content: content) if empty_alt_tag? && !@img.ignore_empty_alt? && !@img.ignore_alt?
 
-      if @img.check_img_http? && @img.scheme == 'http'
-        add_issue("image #{@img.url} uses the http scheme", line: line, content: content)
-      end
+      add_issue("image #{@img.url} uses the http scheme", line: line, content: content) if @img.check_img_http? && @img.scheme == 'http'
     end
 
     external_urls

data/lib/html-proofer/check/links.rb

@@ -35,6 +35,7 @@ class LinkCheck < ::HTMLProofer::Check
         next if @link.allow_missing_href?
         # HTML5 allows dropping the href: http://git.io/vBX0z
         next if @html.internal_subset.name == 'html' && @html.internal_subset.external_id.nil?
+
         add_issue('anchor has no href attribute', line: line, content: content)
         next
       end
@@ -47,9 +48,10 @@ class LinkCheck < ::HTMLProofer::Check
         # we need to skip these for now; although the domain main be valid,
         # curl/Typheous inaccurately return 404s for some links. cc https://git.io/vyCFx
         next if @link.respond_to?(:rel) && @link.rel == 'dns-prefetch'
+
         add_to_external_urls(@link.href)
         next
-      elsif !@link.internal? && !@link.exists?
+      elsif @link.internal? && !@link.exists?
         add_issue("internally linking to #{@link.href}, which does not exist", line: line, content: content)
       end
 
@@ -74,6 +76,7 @@ class LinkCheck < ::HTMLProofer::Check
       handle_tel(link, line, content)
     when 'http'
       return unless @options[:enforce_https]
+
       add_issue("#{link.href} is not an HTTPS link", line: line, content: content)
     end
   end
@@ -103,9 +106,7 @@ class LinkCheck < ::HTMLProofer::Check
       add_issue("trying to find hash of #{link.href}, but #{link.absolute_path} does not exist", line: line, content: content)
     else
       target_html = create_nokogiri link.absolute_path
-      unless hash_check target_html, link.hash
-        add_issue("linking to #{link.href}, but #{link.hash} does not exist", line: line, content: content)
-      end
+      add_issue("linking to #{link.href}, but #{link.hash} does not exist", line: line, content: content) unless hash_check target_html, link.hash
     end
   end
 
@@ -135,6 +136,7 @@ class LinkCheck < ::HTMLProofer::Check
 
   def check_sri(line, content)
     return unless SRI_REL_TYPES.include?(@link.rel)
+
     if !defined?(@link.integrity) && !defined?(@link.crossorigin)
       add_issue("SRI and CORS not provided in: #{@link.src}", line: line, content: content)
     elsif !defined?(@link.integrity)

data/lib/html-proofer/check/opengraph.rb

@@ -1,4 +1,3 @@
-# encoding: utf-8
 # frozen_string_literal: true
 
 class OpenGraphElement < ::HTMLProofer::Element

data/lib/html-proofer/configuration.rb

@@ -65,19 +65,19 @@ module HTMLProofer
     end
 
     def self.parse_json_option(option_name, config)
-      raise ArgumentError.new('Must provide an option name in string format.') unless option_name.is_a?(String)
-      raise ArgumentError.new('Must provide an option name in string format.') unless !option_name.strip.empty?
+      raise ArgumentError, 'Must provide an option name in string format.' unless option_name.is_a?(String)
+      raise ArgumentError, 'Must provide an option name in string format.' if option_name.strip.empty?
 
       return {} if config.nil?
 
-      raise ArgumentError.new('Must provide a JSON configuration in string format.') unless config.is_a?(String)
+      raise ArgumentError, 'Must provide a JSON configuration in string format.' unless config.is_a?(String)
 
       return {} if config.strip.empty?
 
       begin
         JSON.parse(config)
-      rescue
-        raise ArgumentError.new("Option '" + option_name + "' did not contain valid JSON.")
+      rescue StandardError
+        raise ArgumentError, "Option '" + option_name + "' did not contain valid JSON."
       end
     end
   end

data/lib/html-proofer/element.rb

@@ -18,7 +18,7 @@ module HTMLProofer
         instance_variable_set("@#{name}", value.value)
       end
 
-      @aria_hidden = (defined?(@aria_hidden) && @aria_hidden == 'true') ? true : false
+      @aria_hidden = defined?(@aria_hidden) && @aria_hidden == 'true' ? true : false
 
       @data_proofer_ignore = defined?(@data_proofer_ignore)
 
@@ -56,9 +56,11 @@ module HTMLProofer
 
     def url
       return @url if defined?(@url)
+
       @url = (@src || @srcset || @href || '').delete("\u200b").strip
       @url = Addressable::URI.join(base.attr('href') || '', url).to_s if base
       return @url if @check.options[:url_swap].empty?
+
       @url = swap(@url, @check.options[:url_swap])
     end
 
@@ -77,11 +79,11 @@ module HTMLProofer
     end
 
     def hash
-      parts.fragment unless parts.nil?
+      parts&.fragment
     end
 
     def scheme
-      parts.scheme unless parts.nil?
+      parts&.scheme
     end
 
     # path is to an external server
@@ -137,9 +139,22 @@ module HTMLProofer
       !internal?
     end
 
-    # path is an anchor or a query
     def internal?
-      hash_link || param_link || slash_link
+      relative_link? || internal_absolute_link?
+    end
+
+    def internal_absolute_link?
+      url.start_with?('/')
+    end
+
+    def relative_link?
+      return false if remote?
+
+      hash_link || param_link || url.start_with?('.') || url =~ /^\w/
+    end
+
+    def link_points_to_same_page?
+      hash_link || param_link
     end
 
     def hash_link
@@ -150,21 +165,20 @@ module HTMLProofer
       url.start_with?('?')
     end
 
-    def slash_link
-      url.start_with?('|')
-    end
-
     def file_path
-      return if path.nil?
+      return if path.nil? || path.empty?
 
       path_dot_ext = ''
 
-      if @check.options[:assume_extension]
-        path_dot_ext = path + @check.options[:extension]
-      end
+      path_dot_ext = path + @check.options[:extension] if @check.options[:assume_extension]
 
       if path =~ %r{^/} # path relative to root
-        base = File.directory?(@check.src) ? @check.src : File.dirname(@check.src)
+        if File.directory?(@check.src)
+          base = @check.src
+        else
+          root_dir = @check.options[:root_dir]
+          base = root_dir || File.dirname(@check.src)
+        end
       elsif File.exist?(File.expand_path(path, @check.src)) || File.exist?(File.expand_path(path_dot_ext, @check.src)) # relative links, path is a file
         base = File.dirname @check.path
       elsif File.exist?(File.join(File.dirname(@check.path), path)) || File.exist?(File.join(File.dirname(@check.path), path_dot_ext)) # relative links in nested dir, path is a file
@@ -174,7 +188,6 @@ module HTMLProofer
       end
 
       file = File.join base, path
-
       if @check.options[:assume_extension] && File.file?("#{file}#{@check.options[:extension]}")
         file = "#{file}#{@check.options[:extension]}"
       elsif File.directory?(file) && !unslashed_directory?(file) # implicit index support
@@ -187,6 +200,7 @@ module HTMLProofer
     # checks if a file exists relative to the current pwd
     def exists?
       return @checked_paths[absolute_path] if @checked_paths.key? absolute_path
+
       @checked_paths[absolute_path] = File.exist? absolute_path
     end
 
@@ -221,9 +235,9 @@ module HTMLProofer
 
     def html
       # If link is on the same page, then URL is on the current page so can use the same HTML as for current page
-      if (hash_link || param_link) && internal?
+      if link_points_to_same_page?
         @html
-      elsif slash_link && internal?
+      elsif relative_link?
         # link on another page, e.g. /about#Team - need to get HTML from the other page
         create_nokogiri(absolute_path)
       end

data/lib/html-proofer/issue.rb

@@ -56,9 +56,7 @@ module HTMLProofer
           @logger.log :error, "  *  #{issue}"
         else
           msg = "  *  #{issue.send(second_report)}#{issue.line}"
-          if !issue.content.nil? && !issue.content.empty?
-            msg = "#{msg}\n     #{issue.content}"
-          end
+          msg = "#{msg}\n     #{issue.content}" if !issue.content.nil? && !issue.content.empty?
           @logger.log(:error, msg)
         end
       end

data/lib/html-proofer/middleware.rb

@@ -2,6 +2,7 @@
 
 module HTMLProofer
   class Middleware
+    include HTMLProofer::Utils
 
     class InvalidHtmlError < StandardError
       def initialize(failures)
@@ -9,18 +10,18 @@ module HTMLProofer
       end
 
       def message
-          "HTML Validation errors (skip by adding `?proofer-ignore` to URL): \n#{@failures.join("\n")}"
+        "HTML Validation errors (skip by adding `?proofer-ignore` to URL): \n#{@failures.join("\n")}"
       end
     end
 
     def self.options
       @options ||= {
-        type:                :file,
-        allow_missing_href:  true, # Permitted in html5
-        allow_hash_href:     true,
+        type: :file,
+        allow_missing_href: true, # Permitted in html5
+        allow_hash_href: true,
         check_external_hash: true,
-        check_html:          true,
-        url_ignore:          [/.*/], # Don't try to check local files exist
+        check_html: true,
+        url_ignore: [/.*/] # Don't try to check local files exist
       }
     end
 
@@ -46,20 +47,21 @@ module HTMLProofer
       '<BR',
       '<P',
       '<!--'
-    ]
+    ].freeze
 
     def call(env)
       result = @app.call(env)
       return result if env['REQUEST_METHOD'] != 'GET'
       return result if env['QUERY_STRING'] =~ /proofer-ignore/
       return result if result.first != 200
+
       body = []
       result.last.each { |e| body << e }
 
       body = body.join('')
       begin
         html = body.lstrip
-      rescue
+      rescue StandardError
         return result # Invalid encoding; it's not gonna be html.
       end
       if HTML_SIGNATURE.any? { |sig| html.upcase.start_with? sig }
@@ -67,12 +69,10 @@ module HTMLProofer
           'response',
           Middleware.options
         ).check_parsed(
-          Nokogiri::HTML(Utils.clean_content(html)), 'response'
+          Nokogiri::HTML(clean_content(html)), 'response'
         )
 
-        if parsed[:failures].length > 0
-          raise InvalidHtmlError.new(parsed[:failures])
-        end
+        raise InvalidHtmlError, parsed[:failures] unless parsed[:failures].empty?
       end
       result
     end

data/lib/html-proofer/runner.rb

@@ -103,9 +103,7 @@ module HTMLProofer
           check = Object.const_get(klass).new(src, path, html, @options)
           check.run
           external_urls = check.external_urls
-          if @options[:url_swap]
-            external_urls = Hash[check.external_urls.map { |url, file| [swap(url, @options[:url_swap]), file] }]
-          end
+          external_urls = Hash[check.external_urls.map { |url, file| [swap(url, @options[:url_swap]), file] }] if @options[:url_swap]
           result[:external_urls].merge!(external_urls)
           result[:failures].concat(check.issues)
         end
@@ -148,6 +146,7 @@ module HTMLProofer
 
     def checks
       return @checks if defined?(@checks) && !@checks.nil?
+
       @checks = HTMLProofer::Check.subchecks.map(&:name)
       @checks.delete('FaviconCheck') unless @options[:check_favicon]
       @checks.delete('HtmlCheck') unless @options[:check_html]
@@ -159,6 +158,7 @@ module HTMLProofer
     def failed_tests
       result = []
       return result if @failures.empty?
+
       @failures.each { |f| result << f.to_s }
       result
     end

data/lib/html-proofer/url_validator.rb

@@ -36,6 +36,7 @@ module HTMLProofer
 
     def remove_query_values
       return nil if @external_urls.nil?
+
       paths_with_queries = {}
       iterable_external_urls = @external_urls.dup
       @external_urls.each_key do |url|
@@ -46,6 +47,7 @@ module HTMLProofer
                 nil
               end
         next if uri.nil? || uri.query.nil?
+
         iterable_external_urls.delete(url) unless new_url_query_values?(uri, paths_with_queries)
       end
       iterable_external_urls
@@ -108,9 +110,9 @@ module HTMLProofer
       external_urls.each_pair do |url, filenames|
         url = begin
                  clean_url(url)
-               rescue URI::Error, Addressable::URI::InvalidURIError
-                 add_external_issue(filenames, "#{url} is an invalid URL")
-                 next
+              rescue URI::Error, Addressable::URI::InvalidURIError
+                add_external_issue(filenames, "#{url} is an invalid URL")
+                next
                end
 
         method = if hash?(url) && @options[:check_external_hash]
@@ -146,20 +148,18 @@ module HTMLProofer
       response_code = response.code
       response.body.gsub!("\x00", '')
 
-      if filenames.nil?
-        debug_msg = "Received a #{response_code} for #{href}"
-      else
-        debug_msg = "Received a #{response_code} for #{href}  in #{filenames.join(' ')}"
-      end
+      debug_msg = if filenames.nil?
+                    "Received a #{response_code} for #{href}"
+                  else
+                    "Received a #{response_code} for #{href}  in #{filenames.join(' ')}"
+                  end
 
       @logger.log :debug, debug_msg
 
       return if @options[:http_status_ignore].include?(response_code)
 
       if response_code.between?(200, 299)
-        unless check_hash_in_2xx_response(href, effective_url, response, filenames)
-          @cache.add(href, filenames, response_code)
-        end
+        @cache.add(href, filenames, response_code) unless check_hash_in_2xx_response(href, effective_url, response, filenames)
       elsif response.timed_out?
         handle_timeout(href, filenames, response_code)
       elsif response_code.zero?
@@ -168,6 +168,7 @@ module HTMLProofer
         queue_request(:get, href, filenames)
       else
         return if @options[:only_4xx] && !response_code.between?(400, 499)
+
         # Received a non-successful http response.
         msg = "External link #{href} failed: #{response_code} #{response.return_message}"
         add_external_issue(filenames, msg, response_code)
@@ -191,9 +192,7 @@ module HTMLProofer
         xpath << [%(//*[@name="user-content-#{hash}"]|//*[@id="user-content-#{hash}"])]
         # when linking to a file on GitHub, like #L12-L34, only the first "L" portion
         # will be identified as a linkable portion
-        if hash =~ /\A(L\d)+/
-          xpath << [%(//td[@id="#{Regexp.last_match[1]}"])]
-        end
+        xpath << [%(//td[@id="#{Regexp.last_match[1]}"])] if hash =~ /\A(L\d)+/
       end
 
       return unless body_doc.xpath(xpath.join('|')).empty?
@@ -208,6 +207,7 @@ module HTMLProofer
       msg = "External link #{href} failed: got a time out (response code #{response_code})"
       @cache.add(href, filenames, 0, msg)
       return if @options[:only_4xx]
+
       add_external_issue(filenames, msg, response_code)
     end
 
@@ -218,6 +218,7 @@ module HTMLProofer
              Either way, the return message (if any) from the server is: #{return_message}"
       @cache.add(href, filenames, 0, msg)
       return if @options[:only_4xx]
+
       add_external_issue(filenames, msg, response_code)
     end
 

data/lib/html-proofer/utils.rb

@@ -17,7 +17,6 @@ module HTMLProofer
 
       Nokogiri::HTML(clean_content(content))
     end
-    module_function :create_nokogiri
 
     def swap(href, replacement)
       replacement.each do |link, replace|
@@ -25,7 +24,6 @@ module HTMLProofer
       end
       href
     end
-    module_function :swap
 
     # address a problem with Nokogiri's parsing URL entities
     # problem from http://git.io/vBYU1
@@ -35,6 +33,5 @@ module HTMLProofer
         url.gsub(/&(?!amp;)/, '&')
       end
     end
-    module_function :clean_content
   end
 end

data/lib/html-proofer/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HTMLProofer
-  VERSION = '3.13.0'.freeze
+  VERSION = '3.14.0'
 end

metadata

@@ -1,59 +1,59 @@
 --- !ruby/object:Gem::Specification
 name: html-proofer
 version: !ruby/object:Gem::Version
-  version: 3.13.0
+  version: 3.14.0
 platform: ruby
 authors:
 - Garen Torikian
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-09-25 00:00:00.000000000 Z
+date: 2019-11-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: mercenary
+  name: addressable
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.3'
+        version: '2.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.3'
+        version: '2.3'
 - !ruby/object:Gem::Dependency
-  name: nokogiri
+  name: mercenary
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.10'
+        version: '0.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.10'
+        version: '0.3'
 - !ruby/object:Gem::Dependency
-  name: rainbow
+  name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '1.10'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '1.10'
 - !ruby/object:Gem::Dependency
-  name: typhoeus
+  name: parallel
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
@@ -67,21 +67,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
-  name: yell
+  name: rainbow
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
-  name: parallel
+  name: typhoeus
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
@@ -95,21 +95,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
-  name: addressable
+  name: yell
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.3'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.3'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
-  name: redcarpet
+  name: awesome_print
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -123,7 +123,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: codecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -137,7 +137,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop-standard
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -151,7 +151,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop-performance
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -165,7 +165,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: codecov
+  name: redcarpet
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -193,7 +193,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3.1'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -207,7 +207,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: pry-byebug
+  name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -221,7 +221,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: awesome_print
+  name: rubocop-standard
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -235,33 +235,33 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: vcr
+  name: timecop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.9'
+        version: '0.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.9'
+        version: '0.8'
 - !ruby/object:Gem::Dependency
-  name: timecop
+  name: vcr
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: '2.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: '2.9'
 description: Test your rendered HTML files to make sure they're accurate.
 email:
 - gjtorikian@gmail.com