danger-chikuwatter 0.0.3 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e7a602ef5810a9cfe876817c4c1b746266d715fd1af8a0928952488c91171426
-  data.tar.gz: 3aadfb9008e6a0703436a87dccf63737bfc4a45b99e737a7c66a32a6bb61fc23
+  metadata.gz: c734434700698b98637fd16e866f41c4655284027fd4357cd99b89f873d40c90
+  data.tar.gz: 9804b717d23c2fa43b6fd4227ef2881c4dd8ff9efddb2a39dc60795e43ea0065
 SHA512:
-  metadata.gz: 6a752695bba8e31fb3f379550cd2a85da512916155e66b469eb7177ac800c62e3fd6c55000d81d9bf25312142174e3638928d73ae4e6d358e31bddaf140af0e5
-  data.tar.gz: c640f16976cdee35887a14680c8ce1c33ccb464b327accd0ba4e0e3f1f3f48ad190453a4e14b88fc266a6716ec2f04db935020ef9d0463187f6fcbaa2e5570fc
+  metadata.gz: 867ec7780654354f8ca8534852d7743f3e97ca564c0f2b653119cd30125095b08ea1fa1abcccb71b0a19e4640b8013be421f987179c430a30ec2342a5393ae0a
+  data.tar.gz: 7fcdb89fb107a814489ca353416addb22557b7e0992089a6f61ba1fc14e4b062e5cb1fb9abef431fee56970c8cf3868b9873e8ea47c838cad7a07e8310833704

data/README.md

@@ -12,18 +12,36 @@ gem danger-chikuwa
 
 ## Usage
 
+### Report flutter analyze errors and warnings
 Run flutter analyze and save the result to a file.
 
 ```
-$ flutter analyze 2>&1 | tee result.log
+$ flutter analyze 2>&1 | tee analyze.log
 ```
 
 Add the following to your Dangerfile.
 
 ```
-chikuwatter.report "analyze.log"
+chikuwatter.analyze "analyze.log"
+chikuwatter.report
 ```
 
+### Repost riverpod_lint errors and warnings
+Run riverpod_lint and save the result to a file.
+
+```
+$ dart run custom_lint 2>&1 | tee riverpod_lint.log
+```
+
+Add the following to your Dangerfile.
+
+```
+chikuwatter.riverpod_lint_log "riverpod_lint.log"
+chikuwatter.report
+```
+
+### Report inline comments
+
 If `inline_mode` is true, the plugin will report the errors, warnings and **info** as inline comments.
 
 ```

data/lib/chikuwatter/gem_version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Chikuwatter
-  VERSION = "0.0.3"
+  VERSION = "1.0.0"
 end

data/lib/chikuwatter/plugin.rb

@@ -14,7 +14,7 @@ module Danger
   #          chikuwatter.report = result.log
   #
   class DangerChikuwatter < Plugin
-    attr_accessor :project_root, :inline_mode
+    attr_accessor :project_root, :inline_mode, :analyze_log, :riverpod_lint_log
 
     # Project root directory
     def _project_root
@@ -28,14 +28,40 @@ module Danger
       @inline_mode || false
     end
 
+    # Analyze log file path
+    def _analyze_log
+      @analyze_log
+    end
+
+    # Riverpod lint log file path
+    def _riverpod_lint_log
+      @riverpod_lint_log
+    end
+
     # Report analyze results
-    def report(file_path)
-      if File.exist?(file_path)
-        results = parse_analyze_log(file_path)
-        send_reports(results)
-      else
-        fail "analyze log file not found"
+    def report
+      results = []
+      if !_analyze_log && !_riverpod_lint_log
+        fail "You must set report file path."
+      end
+
+      if _analyze_log
+        if File.exist?(_analyze_log)
+          results += parse_analyze_log(_analyze_log)
+        else
+          fail "File not found: #{_analyze_log}"
+        end
       end
+
+      if _riverpod_lint_log
+        if File.exist?(_riverpod_lint_log)
+          results += parse_riverpod_lint_log(_riverpod_lint_log)
+        else
+          fail "File not found: #{_riverpod_lint_log}"
+        end
+      end
+
+      send_reports(results)
     end
 
     # Output level
@@ -91,6 +117,28 @@ module Danger
       return report_data
     end
 
+    # Parse riverpod_lint log
+    def parse_riverpod_lint_log(file_path)
+      _project_root = Dir.pwd
+      report_data = []
+      File.foreach(file_path) do |line|
+        logs = line.split("•")
+        # lib/presentation/screens/favorite_contents/favorite_contents_screen_view_model.dart:135:20 • Generated providers should only depend on other generated providers. Failing to do so may break rules such as "provider_dependencies". • avoid_manual_providers_as_generated_provider_dependency
+        if logs.length < 3
+          next
+        end
+
+        file, line_num = logs[0].strip!.split(":")
+        message = "riverpod_lint • `#{logs[2].strip!}`\n#{logs[1].strip!}"
+
+        report_data.push(
+          ReportData.new(message, Type::WARN, file, line_num.to_i)
+        )
+      end
+
+      return report_data
+    end
+
     # Send analyze results
     def send_reports(results)
       results.each do |data|

data/spec/chikuwatter_spec.rb

@@ -25,18 +25,39 @@ module Danger
       # Some examples for writing tests
       # You should replace these with your own.
 
-      it "Errors on file not found" do
-        @my_plugin.report "no file"
-        expect(@dangerfile.status_report[:errors]).to eq(["analyze log file not found"])
+      it "Errors on analyze file not found" do
+        @my_plugin.analyze_log = "no file"
+        @my_plugin.report
+        expect(@dangerfile.status_report[:errors]).to eq(["File not found: no file"])
       end
 
-      it "Report" do
+      it "Errors on riverpod_lint file not found" do
+        @my_plugin.riverpod_lint_log = "no file"
+        @my_plugin.report
+        expect(@dangerfile.status_report[:errors]).to eq(["File not found: no file"])
+      end
+
+      it "Errors on missing parameters" do
+        @my_plugin.report
+        expect(@dangerfile.status_report[:errors]).to eq(["You must set report file path."])
+      end
+
+      it "Report Analyze results" do
         file_path = "#{File.dirname(__FILE__)}/fixtures/analyze.log"
         @my_plugin.inline_mode = true
-        @my_plugin.report file_path
+        @my_plugin.analyze_log = file_path
+        @my_plugin.report
         expect(@dangerfile.status_report[:warnings]).to eq(["warning • `invalid_null_aware_operator`\nThe receiver can't be null, so the null-aware operator '?.' is unnecessary", "info • `deprecated_member_use`\n'headline1' is deprecated and shouldn't be used. Use displayLarge instead. This feature was deprecated after v3.1.0-0.0.pre"])
         expect(@dangerfile.status_report[:errors]).to eq(["error • `argument_type_not_assignable`\nThe argument type 'String?' can't be assigned to the parameter type 'String'"])
       end
+
+      it "Report Riverpod lint results" do
+        file_path = "#{File.dirname(__FILE__)}/fixtures/riverpod_lint.log"
+        @my_plugin.inline_mode = true
+        @my_plugin.riverpod_lint_log = file_path
+        @my_plugin.report
+        expect(@dangerfile.status_report[:warnings]).to eq(["riverpod_lint • `avoid_manual_providers_as_generated_provider_dependency`\nGenerated providers should only depend on other generated providers. Failing to do so may break rules such as \"provider_dependencies\"."])
+      end
     end
   end
 end

data/spec/fixtures/riverpod_lint.log

@@ -0,0 +1 @@
+lib/main.dart:20:20 • Generated providers should only depend on other generated providers. Failing to do so may break rules such as "provider_dependencies". • avoid_manual_providers_as_generated_provider_dependency

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-chikuwatter
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 1.0.0
 platform: ruby
 authors:
 - watanavex
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-04-11 00:00:00.000000000 Z
+date: 2023-07-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api
@@ -173,6 +173,7 @@ files:
 - lib/danger_plugin.rb
 - spec/chikuwatter_spec.rb
 - spec/fixtures/analyze.log
+- spec/fixtures/riverpod_lint.log
 - spec/spec_helper.rb
 homepage: https://github.com/watanavex/danger-chikuwatter
 licenses:
@@ -200,4 +201,5 @@ summary: A longer description of danger-chikuwatter.
 test_files:
 - spec/chikuwatter_spec.rb
 - spec/fixtures/analyze.log
+- spec/fixtures/riverpod_lint.log
 - spec/spec_helper.rb