rubomatic-html 0.2.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e2feb228bfcfe89977098ed0bcbd954f01d5c98d5463efef59a1d68b100b5562
-  data.tar.gz: 49d1172eeadffa303d8638050429a5c2ae8c39e4844df5a5ae390c91efb11785
+  metadata.gz: 24500e8be719d33394abf55f3acb746181fb5507fc32e4084bbb09c17dff2b7f
+  data.tar.gz: b029dfee3fcec76a9a96e45a74f81fa63ce8440fcb130c060a1763a99d704bfc
 SHA512:
-  metadata.gz: e606e10f3f7d0b03cb07eaac70f1c74af5c211a96b0b8e4ef5f5456513cf851e32cb6ba1f9dc21e35867d6884b1126cd8efc4406862f38473887494a6c5f0d92
-  data.tar.gz: 1934e0c133299678113e12cd919b2513fb62f659bb0f26a944072b6c4641448a2df48327fec1ba090ee267ed65b9459b366d94d5390e6459afb28e476b4b372d
+  metadata.gz: 7c80e7b5db8754f5ce084d2fd0a888e15e8c60d3069fd21d5f1ce90443d4984410bbeb0dbf814f86a3c5443bcea99de07f812b47cf41762763ba318b96951541
+  data.tar.gz: ff966b0a358954f84a3345c358969110bc06d9797eac5e724f601f51a7b16fca75514a25ffecdf615072687be8ce8d999f7d7f250fbe10eae11730872ceff3bd

data/README.adoc

@@ -1,6 +1,8 @@
 = RubomaticHtml
 
-Gem for shared html linting for BrandsInsurance
+Gem for shared html linting for BrandsInsurance.
+
+Make sure ran from project root
 
 == Installation
 

data/exe/rubomatic-html

@@ -1,3 +1,24 @@
 #!/usr/bin/env ruby
 
-puts 1
+require 'rubomatic-html'
+require 'optparse'
+
+options = {}
+parser = OptionParser.new do |parser|
+  parser.on(
+    '-l',
+    '--linting-files comma,separated,list,of,files',
+    Array,
+    'List of files to lint. Defaults to check everything in app/views'
+  )
+
+  parser.on(
+    '-i',
+    '--ignored-files comma,separated,list,of,files',
+    Array,
+    'List of files to ignore. Additionally ignores contents of .lint_html_ignore'
+  )
+end
+parser.parse!(into: options)
+
+RubomaticHtml::Runner.new(options[:'linting-files'], options.fetch(:'ignored-files', [])).run

data/lib/rubomatic-html/cop/base.rb

@@ -5,27 +5,22 @@ module RubomaticHtml
     class Base
       # @return [String]
       attr_accessor :file
-      # @return [String]
-      attr_accessor :line
-      # @return [Integer]
-      attr_accessor :index
 
       # :nodoc:
-      def initialize(file, line, index)
+      def initialize(file)
         @file = file
-        @line = line
-        @index = index
-
-        check_line
       end
 
       private
 
         # Outputs filename:line_number locations of HTML files that trigger the cop
         #
+        # @param line [String] the line in the html
+        # @param index [Integer] the 1-index of the line
+        #
         # @return [void]
         #
-        def check_line
+        def run_for_line(line, index)
         end
     end
   end

data/lib/rubomatic-html/cop/cops.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+require_relative 'base'
+require_relative 'layout/line_length'
+require_relative 'layout/trailing_whitespace'
+require_relative 'style/partial_instance_variable'

data/lib/rubomatic-html/cop/layout/line_length.rb

@@ -5,7 +5,7 @@ module RubomaticHtml
     module Layout
       class LineLength < RubomaticHtml::Cop::Base
         # @see super
-        def check_line
+        def run_for_line(line, index)
           return if line.size <= 120
 
           puts("#{file}:#{index}: is over 120 characters")

data/lib/rubomatic-html/cop/layout/multiple_line_breaks.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+module RubomaticHtml
+  module Cop
+    module Layout
+      class MultipleLineBreaks < RubomaticHtml::Cop::Base
+        # @return [Boolean]
+        attr_accessor :prev_break
+
+        def initialize(file)
+          super
+
+          @prev_break = false
+        end
+
+        # @see super
+        def run_for_line(line, index)
+          if prev_break && line.size.zero?
+            puts("#{file}:#{index}: has multiple line breaks")
+          elsif line.size.zero?
+            @prev_break = true
+          else
+            @prev_break = false
+          end
+        end
+      end
+    end
+  end
+end

data/lib/rubomatic-html/cop/layout/trailing_whitespace.rb

@@ -5,7 +5,7 @@ module RubomaticHtml
     module Layout
       class TrailingWhitespace < RubomaticHtml::Cop::Base
         # @see super
-        def check_line
+        def run_for_line(line, index)
           return unless line.match?(/\s\z/i)
 
           puts("#{file}:#{index}: has trailing whitespace")

data/lib/rubomatic-html/cop/style/partial_instance_variable.rb

@@ -5,7 +5,7 @@ module RubomaticHtml
     module Style
       class PartialInstanceVariable < RubomaticHtml::Cop::Base
         # @see super
-        def check_line
+        def run_for_line(line, index)
           return unless File.basename(file).match?(/^_/i)
           return unless line.match?(/@/i)
 

data/lib/rubomatic-html/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RubomaticHtml
-  VERSION = '0.2.0'
+  VERSION = '0.4.0'
 end

data/lib/rubomatic-html.rb

@@ -0,0 +1,69 @@
+# frozen_string_literal: true
+
+require_relative 'rubomatic-html/version'
+require_relative 'rubomatic-html/cop/cops'
+
+module RubomaticHtml
+  class Runner
+    # @return [Array<String>]
+    attr_accessor :files_to_lint
+    # @return [Array<String>]
+    attr_accessor :files_to_ignore
+
+    # @param linted_files [Array<String>]
+    # @param ignored_files [Array<String>]
+    #
+    def initialize(linted_files, ignored_files)
+      files_to_ignore = Array(ignored_files)
+      ignore_file = '.lint_html_ignore'
+
+      if File.exist?(ignore_file)
+        files_to_ignore += File.readlines(ignore_file, chomp: true)
+      end
+
+      files_to_lint = Array(linted_files)
+
+      if files_to_lint.empty?
+        files_to_lint = Dir[File.join('app', 'views', '**', '*')]
+      end
+
+      @files_to_lint = files_to_lint
+      @files_to_ignore = files_to_ignore
+    end
+
+    def run
+      files_to_lint.each do |file|
+        next if files_to_ignore.any? { |ignored| file.end_with?(ignored) }
+
+        ext = File.extname(file)
+        next if ext.match?(/haml/i)
+
+        check_it = ext.match?(/html/i)
+        check_it ||= ext.match?(/erb\z/i)
+
+        next unless check_it
+
+        run_file(file)
+      end
+    end
+
+    private
+
+      def all_cops
+        @all_cops ||= [
+          RubomaticHtml::Cop::Layout::LineLength,
+          RubomaticHtml::Cop::Layout::MultipleLineBreaks,
+          RubomaticHtml::Cop::Layout::TrailingWhitespace,
+          RubomaticHtml::Cop::Style::PartialInstanceVariable
+        ]
+      end
+
+      def run_file(file)
+        cops = all_cops.map { |cop| cop.new(file) }
+
+        File.open(file).each_line(chomp: true).with_index(1) do |line, index|
+          cops.each { |cop| cop.run_for_line(line, index) }
+        end
+      end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubomatic-html
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Brands Insurance
@@ -24,20 +24,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 2.2.0
-- !ruby/object:Gem::Dependency
-  name: rubocop-brands_insurance
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.3.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.3.0
 description: 
 email:
 - documents@brandsinsurance.com
@@ -50,12 +36,14 @@ files:
 - LICENSE.txt
 - README.adoc
 - exe/rubomatic-html
+- lib/rubomatic-html.rb
 - lib/rubomatic-html/cop/base.rb
+- lib/rubomatic-html/cop/cops.rb
 - lib/rubomatic-html/cop/layout/line_length.rb
+- lib/rubomatic-html/cop/layout/multiple_line_breaks.rb
 - lib/rubomatic-html/cop/layout/trailing_whitespace.rb
 - lib/rubomatic-html/cop/style/partial_instance_variable.rb
 - lib/rubomatic-html/version.rb
-- lib/rubomatic_html.rb
 homepage: https://github.com/BrandsInsurance/expert-chainsaw/
 licenses:
 - MIT

data/lib/rubomatic_html.rb

@@ -1,6 +0,0 @@
-# frozen_string_literal: true
-
-require_relative 'rubomatic-html/version'
-
-module RubomaticHtml
-end