danger-compose_compiler_metrics 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0a4e4fdbd974ca9e26bf31a9d9d458a24701fba13e2cf50294ae0787573931f4
-  data.tar.gz: 88b1ab8dacfa22cb954b930ffa589e8a3066e949d66c2275f93391a1bf829b44
+  metadata.gz: 5174bc2092a2c48fa3a84c6cb7006b0836fc9ccdc55f647efad88cc3fbd52e65
+  data.tar.gz: de4c9958f624409b65e36aa49d5b001de6f474e79c43afe51a42011f1150ac91
 SHA512:
-  metadata.gz: bba795ff85c82cd20464458c407375dfbeeca2bcdc0f42b54cdf1842d3fb6b60507ff24e8b1ae796d753a83eab882b90d7f5e742bddd3d65187675099484c63f
-  data.tar.gz: 53b535436b8654a60e2cbb43a42fa4aec95bae0a8cf684627e6ef40b8a063b14145e18d1560b22ef05e350efa99291a4af2d9e0f5bfb14ba7a5f4d050d00e9e3
+  metadata.gz: 4543a2ecc312efe6d7357acaf70e19bf84d4f2938cc525b9c33db740f04aee929acc7c2cdd17a98c5cb112bb3d3595654ac92498d3bf5de665e246608ea7e3b3
+  data.tar.gz: 77d0f263a40a8286b3b36cc6a3dc2bf6b3dff3370e91717d5bdefd4ed824ddecb067c8f14a9c70c1e8d000a35867817e66895f16701d9f653cef050f86fb251b

data/lib/compose_compiler_metrics/gem_version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ComposeCompilerMetrics
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/lib/compose_compiler_metrics/helper.rb

@@ -45,4 +45,8 @@ module Helper
     </details>
     HTML
   end
+
+  def installed?(command)
+    system("which #{command} > /dev/null")
+  end
 end

data/lib/compose_compiler_metrics/plugin.rb

@@ -10,89 +10,64 @@ module Danger
     include Helper
 
     def report_difference(metrics_dir, base_metrics_dir)
-      markdown('# Compose Compiler Metrics Difference Report')
+      unless installed?("diff")
+        error "diff command not found. Please install diff command."
+        return
+      end
+
+      markdown("# Compose Compiler Metrics Difference Report")
       build_variants(metrics_dir).each do |module_name, build_variant|
         markdown("## #{module_name} - #{build_variant}")
 
         # Metrics Report
         metrics_path = File.join(metrics_dir, metrics_filename(module_name, build_variant))
         base_metrics_path = File.join(base_metrics_dir, metrics_filename(module_name, build_variant))
-        report = `diff -u #{base_metrics_path} #{metrics_path}`
-
-        markdown(
-          folding(
-            '### Metrics',
-            if report.empty?
-              'No difference found.'
-            else
-              <<~MARKDOWN
-              ```diff
-              #{report}
-              ```
-              MARKDOWN
-            end
-          )
-        )
+        report_file_difference("Metrics", metrics_path, base_metrics_path)
 
         # Composable Stats Report
         composable_stats_report_path = File.join(metrics_dir, composable_stats_report_path(module_name, build_variant))
         base_composable_stats_report_path = File.join(base_metrics_dir, composable_stats_report_path(module_name, build_variant))
-        report = `diff -u #{base_composable_stats_report_path} #{composable_stats_report_path}`
-
-        markdown(
-          folding(
-            '### Composable Stats Report',
-            if report.empty?
-              'No difference found.'
-            else
-              <<~MARKDOWN
-              ```diff
-              #{report}
-              ```
-              MARKDOWN
-            end
-          )
-        )
+        report_file_difference("Composable Stats Report", composable_stats_report_path, base_composable_stats_report_path)
 
         # Composable Report
         composable_report_path = File.join(metrics_dir, composable_report_path(module_name, build_variant))
         base_composable_report_path = File.join(base_metrics_dir, composable_report_path(module_name, build_variant))
-        report = `diff -u #{base_composable_report_path} #{composable_report_path}`
-
-        markdown(
-          folding(
-            '### Composable Report',
-            if report.empty?
-              'No difference found.'
-            else
-              <<~MARKDOWN
-              ```diff
-              #{report}
-              ```
-              MARKDOWN
-            end
-          )
-        )
+        report_file_difference("Composable Report", composable_report_path, base_composable_report_path)
 
         # Class Report
         class_report_path = File.join(metrics_dir, class_report_path(module_name, build_variant))
         base_class_report_path = File.join(base_metrics_dir, class_report_path(module_name, build_variant))
-        report = `diff -u #{base_class_report_path} #{class_report_path}`
-        markdown(
-          folding(
-            '### Class Report',
-            if report.empty?
-              'No difference found.'
-            else
-              <<~MARKDOWN
-              ```diff
-              #{report}
-              ```
-              MARKDOWN
-            end
-          )
-        )
+        report_file_difference("Composable Report", class_report_path, base_class_report_path)
+      end
+    end
+
+    def report_file_difference(title, metrics_path, base_metrics_path)
+      unless File.exist?(metrics_path)
+        warn "DangerComposeCompilerMetrics: new file not found at #{metrics_path}. Skipping file difference report."
+        return
+      end
+
+      unless File.exist?(base_metrics_path)
+        warn "DangerComposeCompilerMetrics: reference file not found at #{base_metrics_path}. Skipping file difference report."
+        return
       end
+
+      report = `diff -u #{base_metrics_path} #{metrics_path}`
+
+      markdown(
+        folding(
+          "### #{title}",
+          if report.empty?
+            "No difference found."
+          else
+            <<~MARKDOWN
+            ```diff
+            #{report}
+            ```
+            MARKDOWN
+          end
+        )
+      )
     end
 
     def report(metrics_dir)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-compose_compiler_metrics
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Tomoki Yamashita
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-02-08 00:00:00.000000000 Z
+date: 2024-02-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api