danger-textlint 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e2382053d93e9c0e1eed4097159796a353a3bd9276fa9a44e4d2d407712e6045
-  data.tar.gz: 8d9bbb9847bdb36c5adda08c680deb2eaf48c9a64886a834bb1fed0ee33b207a
+  metadata.gz: 4390839bd6f828e1785379ba7d06040f8c5029602b20d47f2a4480eae6752d4c
+  data.tar.gz: a353be06a739e47e57d23aa790b7e469b7ac8e5d60005f1e0fef175b8cb6fd62
 SHA512:
-  metadata.gz: 19e222aa434bd619b2362c147607ffd6cbd84ca4485358596eba26df6e7331e1f232a8109b800b8931cefa8a7ed79987e6c5e04cf85827fb52fa21bfa0db9698
-  data.tar.gz: 4a34ed18e8280a83de8b755b1ace97cf5ee9fff674f9e2cd8f059ac88dea0341482b5f98755efec0bcd5bc59a69768704b4fbf11eb12905b3560c3662d232e20
+  metadata.gz: 32b9a7b3606c0b0bda6b3e0edc2f4085d00f915a7d76aa3aaa404a7f90f1ed11d72163f5121332eeafea80f2a6bf5d864aa5c24ffbb6ddcf536d889708084939
+  data.tar.gz: 5a728a6b925e76970694144a4ee8f9037fd2a1904ee2fbdcb6524841a11dbd78327632ddbd2065774e90e8709c7975759c357170ce2333f7081dc76fb91702c7

data/.rubocop.yml

@@ -153,3 +153,6 @@ Security/YAMLLoad:
 
 Style/FrozenStringLiteralComment:
   Enabled: false
+
+Metrics/ModuleLength:
+  Enabled: false

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    danger-textlint (1.1.0)
+    danger-textlint (1.2.0)
       danger-plugin-api (~> 1.0)
 
 GEM

data/lib/textlint/gem_version.rb

@@ -1,3 +1,3 @@
 module Textlint
-  VERSION = "1.1.0".freeze
+  VERSION = "1.2.0".freeze
 end

data/lib/textlint/plugin.rb

@@ -14,6 +14,11 @@ module Danger
   #          textlint.max_severity = "warn"
   #          textlint.lint
   #
+  # @example Max inline comment number. If you want disable this feature, please set nil. Default: nil
+  #
+  #          textlint.max_comment_num = 5
+  #          textlint.lint
+  #
   # @see  Kesin11/danger-textlint
   # @tags lint, textlint
   #
@@ -27,6 +32,11 @@ module Danger
     # @return [String]
     attr_accessor :max_severity
 
+    # Set max danger reporting comment number
+    # choice: nil or integer
+    # @return [String]
+    attr_accessor :max_comment_num
+
     # Execute textlint and send comment
     # @return [void]
     def lint
@@ -88,7 +98,13 @@ module Danger
     end
 
     def send_comment(errors)
-      errors.each do |error|
+      limited_errors = errors
+      if max_comment_num && limited_errors.size > max_comment_num
+        limited_errors = limited_errors.first(max_comment_num)
+        send("warn", "Textlint reported more than #{max_comment_num} problems, but danger-textlint doesn't to display all problems. Please run textlint in your machine and check all problems.")
+      end
+
+      limited_errors.each do |error|
         send(error[:severity], error[:message], file: error[:file_path], line: error[:line])
       end
     end

data/spec/textlint_spec.rb

@@ -100,6 +100,33 @@ module Danger
           )
         end
       end
+
+      context "with .max_comment_num = 5" do
+        let(:max_comment_num) { 5 }
+        before do
+          @textlint.max_comment_num = max_comment_num
+          @textlint.lint
+        end
+
+        it "status_report" do
+          status_report = @textlint.status_report
+          expect(status_report[:errors].size).to eq(max_comment_num)
+        end
+
+        it "violation_report" do
+          violation_report = @textlint.violation_report
+          expect(violation_report[:errors].size).to eq(max_comment_num)
+        end
+
+        it "danger comment" do
+          # find not inline comment
+          comment = @textlint.violation_report[:warnings].find do |warning|
+            warning.message.match(/Textlint reported more than/)
+          end
+
+          expect(comment).not_to be_nil
+        end
+      end
     end
 
     describe ".target_files" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-textlint
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Kesin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-02-22 00:00:00.000000000 Z
+date: 2018-02-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api