goodcheck 2.4.3 → 2.4.4

data/goodcheck.gemspec

@@ -24,12 +24,13 @@ Gem::Specification.new do |spec|
   spec.required_ruby_version = '>= 2.4.0'
 
   spec.add_development_dependency "bundler", ">= 1.16"
-  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "rake", "~> 13.0"
   spec.add_development_dependency "minitest", "~> 5.0"
-  spec.add_development_dependency "minitest-reporters", "~> 1.3.6"
+  spec.add_development_dependency "minitest-reporters", "~> 1.4.2"
 
   spec.add_runtime_dependency "activesupport", ">= 4.0", "< 7.0"
-  spec.add_runtime_dependency "strong_json", "~> 1.1.0"
+  spec.add_runtime_dependency "strong_json", ">= 1.1", "< 2.2"
   spec.add_runtime_dependency "rainbow", "~> 3.0.0"
   spec.add_runtime_dependency "httpclient", "~> 2.8.3"
+  spec.add_runtime_dependency "psych", ">= 3.1", "< 4.0" # NOTE: Needed for old Ruby versions (<= 2.5)
 end

data/goodcheck.yml

@@ -0,0 +1,10 @@
+import:
+  - https://raw.githubusercontent.com/sider/goodcheck-rules/master/rules/typo.yml
+  - https://raw.githubusercontent.com/sider/goodcheck-rules/master/rules/ruby.yml
+
+exclude:
+  - "**/build"
+  - "**/node_modules"
+  - "**/vendor"
+  - "**/*.{ico,pdf,png}"
+  - "**/yarn.lock"

data/lib/goodcheck/buffer.rb

@@ -6,6 +6,7 @@ module Goodcheck
     def initialize(path:, content:)
       @path = path
       @content = content
+      @line_ranges = nil
     end
 
     def line_ranges

data/lib/goodcheck/cli.rb

@@ -18,7 +18,7 @@ module Goodcheck
       test: "Test your configuration",
       pattern: "Print regexp for rules",
       version: "Print version",
-      help: "Show goodcheck version and quit"
+      help: "Show help and quit"
     }
 
 
@@ -27,8 +27,12 @@ module Goodcheck
 
       if COMMANDS.key?(command)
         __send__(command, args)
+      elsif command == :"--version"
+        version(args)
       else
+        stderr.puts "Invalid command: #{command}" if command
         help(args)
+        1
       end
     rescue => exn
       stderr.puts exn.inspect
@@ -80,7 +84,7 @@ module Goodcheck
       if args.empty?
         targets << Pathname(".")
       else
-        targets.push *args.map {|arg| Pathname(arg) }
+        args.each {|arg| targets << Pathname(arg) }
       end
 
       reporter = case format

data/lib/goodcheck/commands/check.rb

@@ -1,6 +1,8 @@
 module Goodcheck
   module Commands
     class Check
+      DEFAULT_EXCLUSIONS = [".git", ".svn", ".hg"].freeze
+
       attr_reader :config_path
       attr_reader :rules
       attr_reader :targets
@@ -82,24 +84,23 @@ module Goodcheck
         end
       end
 
-      def is_dotfile?(path)
-        /\A\.[^.]+/.match?(path.basename.to_s)
-      end
-
       def each_file(path, immediate: false, &block)
         case
         when path.symlink?
           # noop
         when path.directory?
-          if immediate || (!is_dotfile?(path) && !excluded?(path))
+          case
+          when DEFAULT_EXCLUSIONS.include?(path.basename.to_s)
+            # noop
+          when immediate || !excluded?(path)
             path.children.each do |child|
               each_file(child, &block)
             end
           end
         when path.file?
           case
-          when path == config_path || is_dotfile?(path)
-            # Skip dotfiles/config file unless explicitly given by command line
+          when path == config_path
+            # Skip the config file unless explicitly given by command line
             yield path if immediate
           when excluded?(path)
             # Skip excluded files unless explicitly given by command line

data/lib/goodcheck/commands/config_loading.rb

@@ -5,7 +5,7 @@ module Goodcheck
 
       def load_config!(force_download:, cache_path:)
         import_loader = ImportLoader.new(cache_path: cache_path, force_download: force_download, config_path: config_path)
-        content = JSON.parse(JSON.dump(YAML.load(config_path.read, config_path.to_s)), symbolize_names: true)
+        content = JSON.parse(JSON.dump(YAML.load(config_path.read, filename: config_path.to_s)), symbolize_names: true)
         loader = ConfigLoader.new(path: config_path, content: content, stderr: stderr, import_loader: import_loader)
         @config = loader.load
       end

data/lib/goodcheck/commands/test.rb

@@ -22,6 +22,11 @@ module Goodcheck
         handle_config_errors stderr do
           load_config!(cache_path: cache_dir_path, force_download: force_download)
 
+          if config.rules.empty?
+            stdout.puts "No rules."
+            return 0
+          end
+
           validate_rule_uniqueness or return 1
           validate_rules or return 1
 
@@ -44,7 +49,8 @@ module Goodcheck
           stdout.puts "  OK!👍"
           true
         else
-          stdout.puts(Rainbow("  Found #{duplicated_ids.size} duplications.😞").red)
+          count = duplicated_ids.size
+          stdout.puts(Rainbow("  Found #{count} #{'duplication'.pluralize(count)}.😞").red)
           duplicated_ids.each do |id|
             stdout.puts "    #{id}"
           end
@@ -54,10 +60,13 @@ module Goodcheck
 
       def validate_rules
         test_pass = true
+        success_count = 0
+        failure_count = 0
+        failed_rule_ids = Set[]
 
         config.rules.each do |rule|
           if rule.triggers.any? {|trigger| !trigger.passes.empty? || !trigger.fails.empty?}
-            stdout.puts "Testing rule #{rule.id}..."
+            stdout.puts "Testing rule #{Rainbow(rule.id).cyan}..."
 
             rule_ok = true
 
@@ -82,7 +91,8 @@ module Goodcheck
                   rule_ok = false
 
                   pass_errors.each do |_, index|
-                    stdout.puts "    #{(index+1).ordinalize} pass example matched.😱"
+                    stdout.puts "    #{(index+1).ordinalize} #{Rainbow('pass').green} example matched.😱"
+                    failed_rule_ids << rule.id
                   end
                 end
 
@@ -91,7 +101,8 @@ module Goodcheck
                   rule_ok = false
 
                   fail_errors.each do |_, index|
-                    stdout.puts "    #{(index+1).ordinalize} fail example didn't match.😱"
+                    stdout.puts "    #{(index+1).ordinalize} #{Rainbow('fail').red} example didn't match.😱"
+                    failed_rule_ids << rule.id
                   end
                 end
               end
@@ -104,10 +115,27 @@ module Goodcheck
 
             if rule_ok
               stdout.puts "  OK!🎉"
+              success_count += 1
+            else
+              failure_count += 1
             end
           end
         end
 
+        unless failed_rule_ids.empty?
+          stdout.puts ""
+          stdout.puts "Failed rules:"
+          failed_rule_ids.each do |rule_id|
+            stdout.puts "  - #{Rainbow(rule_id).background(:red)}"
+          end
+        end
+
+        rule_count = success_count + failure_count
+        stdout.puts ""
+        stdout.puts ["Tested #{rule_count} #{'rule'.pluralize(rule_count)}",
+                     Rainbow("#{success_count} #{'success'.pluralize(success_count)}").green,
+                     Rainbow("#{failure_count} #{'failure'.pluralize(failure_count)}").red].join(", ")
+
         test_pass
       end
 

data/lib/goodcheck/config_loader.rb

@@ -241,7 +241,7 @@ module Goodcheck
 
       Goodcheck.logger.tagged import do
         import_loader.load(import) do |content|
-          json = JSON.parse(JSON.dump(YAML.load(content, import)), symbolize_names: true)
+          json = JSON.parse(JSON.dump(YAML.load(content, filename: import)), symbolize_names: true)
 
           Schema.rules.coerce json
           load_rules(rules, json)

data/lib/goodcheck/glob.rb

@@ -9,7 +9,7 @@ module Goodcheck
     end
 
     def test(path)
-      path.fnmatch?(pattern, File::FNM_PATHNAME | File::FNM_EXTGLOB)
+      path.fnmatch?(pattern, File::FNM_PATHNAME | File::FNM_EXTGLOB | File::FNM_DOTMATCH)
     end
 
     def ==(other)

data/lib/goodcheck/issue.rb

@@ -10,6 +10,7 @@ module Goodcheck
       @range = range
       @rule = rule
       @text = text
+      @location = nil
     end
 
     def path

data/lib/goodcheck/trigger.rb

@@ -12,6 +12,8 @@ module Goodcheck
       @passes = passes
       @fails = fails
       @negated = negated
+      @by_pattern = false
+      @skips_fail_examples = false
     end
 
     def by_pattern!

data/lib/goodcheck/version.rb

@@ -1,3 +1,3 @@
 module Goodcheck
-  VERSION = "2.4.3"
+  VERSION = "2.4.4"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: goodcheck
 version: !ruby/object:Gem::Version
-  version: 2.4.3
+  version: 2.4.4
 platform: ruby
 authors:
 - Soutaro Matsumoto
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-11-07 00:00:00.000000000 Z
+date: 2019-11-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -58,14 +58,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.3.6
+        version: 1.4.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.3.6
+        version: 1.4.2
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
@@ -90,16 +90,22 @@ dependencies:
   name: strong_json
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.1'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: '2.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: '1.1'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.2'
 - !ruby/object:Gem::Dependency
   name: rainbow
   requirement: !ruby/object:Gem::Requirement
@@ -128,6 +134,26 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 2.8.3
+- !ruby/object:Gem::Dependency
+  name: psych
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.1'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.1'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4.0'
 description: Regexp based customizable linter
 email:
 - matsumoto@soutaro.com
@@ -175,6 +201,7 @@ files:
 - docusaurus/website/versioned_docs/version-1.0.0/rules.md
 - docusaurus/website/versioned_docs/version-1.0.2/rules.md
 - docusaurus/website/versioned_docs/version-2.4.0/configuration.md
+- docusaurus/website/versioned_docs/version-2.4.3/rules.md
 - docusaurus/website/versioned_sidebars/version-1.0.0-sidebars.json
 - docusaurus/website/versioned_sidebars/version-1.0.2-sidebars.json
 - docusaurus/website/versioned_sidebars/version-2.4.0-sidebars.json
@@ -182,6 +209,7 @@ files:
 - docusaurus/website/yarn.lock
 - exe/goodcheck
 - goodcheck.gemspec
+- goodcheck.yml
 - lib/goodcheck.rb
 - lib/goodcheck/analyzer.rb
 - lib/goodcheck/array_helper.rb
@@ -231,7 +259,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Regexp based customizable linter