danger-periphery 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5b949d4e8279afb04d63616f89e15f3ef0efaf700b83d027273f1973d085fa23
-  data.tar.gz: cd8e2564827d41bdee17e7b8fead197f3dc1481f129ee9dcbc22779ea1783489
+  metadata.gz: 15302ed92faf0a3230ddcb167fe3db0ae2e8c1a51d6551d85a4c2cc37c760dd4
+  data.tar.gz: 18880214efa7cb5a29ee9b6872688a6eeb19d805e07cad6c445afeeb24f4cad2
 SHA512:
-  metadata.gz: 3d0e28054ab01f50603b92cf32b6005d1befa875545afaa9d8fbd80bdf048d097776bcfcdfe01cc746a0cc24d9455284a400be7b9d11142249db9b2731d1b578
-  data.tar.gz: 97d0f89856d52fe03fbda8f3b8539279343e2b28340042ee5db2aa2984222cf6e3ac5eacde6ad17eebc7b393f9d6163ce16f296e8a044fe984442b31566ae6ee
+  metadata.gz: 5e33a6b9fa62d733004fb63337fdff4973dd3b38d40a096db0b84a1249bc7789a3612a99a88d31020e22dd6494c3b7d3c953fe721be16347e825a58e8900a177
+  data.tar.gz: 47a37136e81dc9c4b6fc6b3331e476810e4520503c7fd062714363675926c51525502dc58ef2d610b890ee0d73d731d8ac87910bd8e6ab2f051f47c980b4ac88

data/lib/danger_plugin.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require "periphery"
+require 'periphery'
 
 module Danger
   # Analyze Swift files and detect unused codes in your project.
-  # This is done using [Periphery](https://github.com/peripheryapp/periphery).
+  # This is done using {https://github.com/peripheryapp/periphery Periphery}.
   #
   # @example Specifying options to Periphery.
   #
@@ -15,7 +15,7 @@ module Danger
   #            clean_build: true
   #          )
   #
-  # @see manicmaniac/danger-periphery
+  # @see file:README.md
   # @tags swift
   class DangerPeriphery < Plugin
     # Path to Periphery executable.
@@ -40,44 +40,58 @@ module Danger
 
     OPTION_OVERRIDES = {
       disable_update_check: true,
-      format: "checkstyle",
+      format: 'checkstyle',
       quiet: true
     }.freeze
 
     def initialize(dangerfile)
       super(dangerfile)
-      @postprocessor = ->(path, line, column, message) { true }
+      @postprocessor = ->(_path, _line, _column, _message) { true }
     end
 
     # Scans Swift files.
     # Raises an error when Periphery executable is not found.
     #
+    # @example Ignore all warnings from files matching regular expression
+    #   periphery.scan do |violation|
+    #     ! violation.path.match(/.*\/generated\.swift/)
+    #   end
+    #
     # @param [Hash] options Options passed to Periphery with the following translation rules.
     #                       1. Replace all underscores with hyphens in each key.
     #                       2. Prepend double hyphens to each key.
     #                       3. If value is an array, transform it to comma-separated string.
     #                       4. If value is true, drop value and treat it as option without argument.
-    #                       5. Override some options like --disable-update-check, --format, --quiet and so.
+    #                       5. Override some options listed in {OPTION_OVERRIDES}.
+    #                       Run +$ periphery help scan+ for available options.
+    #
+    # @param [Proc] block   Block to process each warning just before showing it.
+    #                       The Proc receives 1 {Periphery::ScanResult} instance as argument.
+    #                       If the Proc returns falsy value, the warning corresponding to the given ScanResult will be
+    #                       suppressed, otherwise not.
+    #
     # @return [void]
     def scan(**options, &block)
       output = Periphery::Runner.new(binary_path).scan(options.merge(OPTION_OVERRIDES))
       files = files_in_diff
-      Periphery::CheckstyleParser.new.parse(output).
-        lazy.
-        select { |entry| files.include?(entry.path) }.
-        map { |entry| postprocess(entry, &block) }.
-        force.
-        compact.
-        each { |path, line, column, message| warn(message, file: path, line: line) }
+      Periphery::CheckstyleParser.new.parse(output).each do |entry|
+        next unless files.include?(entry.path)
+
+        result = postprocess(entry, &block)
+        next unless result
+
+        path, line, _column, message = result
+        warn(message, file: path, line: line)
+      end
     end
 
-    # Convenience method to set `postprocessor` with block.
+    # Convenience method to set {#postprocessor} with block.
     #
     # @return [Proc]
     #
     # @example Ignore all warnings from files matching regular expression
     #   periphery.process_warnings do |path, line, column, message|
-    #     !path.match(/.*\/generated\.swift/)
+    #     ! path.match(/.*\/generated\.swift/)
     #   end
     def process_warnings(&block)
       @postprocessor = block
@@ -88,26 +102,34 @@ module Danger
     def files_in_diff
       # Taken from https://github.com/ashfurrow/danger-ruby-swiftlint/blob/5184909aab00f12954088684bbf2ce5627e08ed6/lib/danger_plugin.rb#L214-L216
       renamed_files_hash = git.renamed_files.to_h { |rename| [rename[:before], rename[:after]] }
-      post_rename_modified_files = git.modified_files.map { |modified_file| renamed_files_hash[modified_file] || modified_file }
+      post_rename_modified_files = git.modified_files.map do |modified_file|
+        renamed_files_hash[modified_file] || modified_file
+      end
       (post_rename_modified_files - git.deleted_files) + git.added_files
     end
 
     def postprocess(entry, &block)
       if block
-        if block.call(entry)
-          [entry.path, entry.line, entry.column, entry.message]
-        end
+        postprocess_with_block(entry, &block)
+      else
+        postprocess_with_postprocessor(entry)
+      end
+    end
+
+    def postprocess_with_block(entry, &block)
+      [entry.path, entry.line, entry.column, entry.message] if block.call(entry)
+    end
+
+    def postprocess_with_postprocessor(entry)
+      result = @postprocessor.call(entry.path, entry.line, entry.column, entry.message)
+      if !result
+        nil
+      elsif result.is_a?(TrueClass)
+        [entry.path, entry.line, entry.column, entry.message]
+      elsif result.is_a?(Array) && result.size == 4
+        result
       else
-        result = @postprocessor.call(entry.path, entry.line, entry.column, entry.message)
-        if !result
-          nil
-        elsif result.kind_of?(TrueClass)
-          [entry.path, entry.line, entry.column, entry.message]
-        elsif result.kind_of?(Array) && result.size == 4
-          result
-        else
-          raise "Proc passed to postprocessor must return one of nil, true, false and Array that includes 4 elements."
-        end
+        raise 'Proc passed to postprocessor must return one of nil, true, false and Array that includes 4 elements.'
       end
     end
   end

data/lib/periphery/checkstyle_parser.rb

@@ -1,14 +1,15 @@
 # frozen_string_literal: true
 
-require "pathname"
-require "periphery/scan_result"
-require "rexml/parsers/baseparser"
-require "rexml/parsers/streamparser"
-require "rexml/streamlistener"
+require 'pathname'
+require 'periphery/scan_result'
+require 'rexml/parsers/streamparser'
+require 'rexml/streamlistener'
 
 module Periphery
+  # Parses {https://checkstyle.sourceforge.io/ Checkstyle} format XML produced by Periphery with
+  # +--format=checkstyle+ option.
   class CheckstyleParser
-    class Listener
+    class Listener # :nodoc:
       include REXML::StreamListener
 
       attr_reader :results
@@ -20,22 +21,17 @@ module Periphery
 
       def tag_start(name, attrs)
         case name
-        when "file"
-          @current_file = relative_path(attrs["name"])
-        when "error"
+        when 'file'
+          @current_file = relative_path(attrs['name'])
+        when 'error'
           if @current_file
-            @results << ScanResult.new(
-              @current_file,
-              attrs["line"].to_i,
-              attrs["column"].to_i,
-              attrs["message"]
-            )
+            @results << ScanResult.new(@current_file, attrs['line'].to_i, attrs['column'].to_i, attrs['message'])
           end
         end
       end
 
       def tag_end(name)
-        @current_file = nil if name == "file"
+        @current_file = nil if name == 'file'
       end
 
       private

data/lib/periphery/runner.rb

@@ -1,35 +1,32 @@
 # frozen_string_literal: true
 
-require "open3"
+require 'open3'
 
 module Periphery
-  class Runner
+  class Runner # :nodoc:
     attr_reader :binary_path
 
     def initialize(binary_path)
-      @binary_path = binary_path || "periphery"
+      @binary_path = binary_path || 'periphery'
     end
 
     def scan(options)
-      arguments = [binary_path, "scan"] + scan_arguments(options)
+      arguments = [binary_path, 'scan'] + scan_arguments(options)
       stdout, stderr, status = Open3.capture3(*arguments)
-      if status.success?
-        stdout
-      else
-        raise "error: #{arguments} exited with status code #{status.exitstatus}. #{stderr}" unless status.success?
-      end
+      raise "error: #{arguments} exited with status code #{status.exitstatus}. #{stderr}" unless status.success?
+
+      stdout
     end
 
     def scan_arguments(options)
-      options.
-        lazy.
-        select { |_key, value| value }.
-        map { |key, value| value.kind_of?(TrueClass) ? [key, nil] : [key, value] }.
-        map { |key, value| value.kind_of?(Array) ? [key, value.join(",")] : [key, value] }.
-        map { |key, value| ["--#{key.to_s.tr('_', '-')}", value&.to_s] }.
-        force.
-        flatten.
-        compact
+      options.each_with_object([]) do |(key, value), new_options|
+        next unless value
+
+        value = nil if value.is_a?(TrueClass)
+        value = value.join(',') if value.is_a?(Array)
+        new_options << "--#{key.to_s.tr('_', '-')}"
+        new_options << value&.to_s if value
+      end
     end
   end
 end

data/lib/periphery.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 
-require "periphery/runner"
-require "periphery/checkstyle_parser"
+require 'periphery/runner'
+require 'periphery/checkstyle_parser'

data/lib/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DangerPeriphery
-  VERSION = "0.2.0"
+  VERSION = '0.2.1'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-periphery
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Ryosuke Ito
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-07-22 00:00:00.000000000 Z
+date: 2022-07-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api
@@ -27,39 +27,16 @@ dependencies:
 description: A Danger plugin to detect unused codes.
 email:
 - rito.0305@gmail.com
-executables:
-- download_periphery
+executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".github/workflows/lint.yml"
-- ".github/workflows/test.yml"
-- ".gitignore"
-- ".rspec"
-- ".rubocop.yml"
-- Dangerfile
-- Gemfile
-- Gemfile.lock
-- Guardfile
-- LICENSE.txt
-- README.md
-- Rakefile
-- bin/download_periphery
-- danger-periphery.gemspec
 - lib/danger_plugin.rb
 - lib/periphery.rb
 - lib/periphery/checkstyle_parser.rb
 - lib/periphery/runner.rb
 - lib/periphery/scan_result.rb
 - lib/version.rb
-- spec/danger_plugin_spec.rb
-- spec/periphery/checkstyle_parser_spec.rb
-- spec/periphery/runner_spec.rb
-- spec/spec_helper.rb
-- spec/support/fixtures/github_pr.json
-- spec/support/fixtures/mock-periphery
-- spec/support/fixtures/test.xcodeproj/project.pbxproj
-- spec/support/fixtures/test/main.swift
 homepage: https://github.com/manicmaniac/danger-periphery
 licenses:
 - MIT

data/.github/workflows/lint.yml

@@ -1,12 +0,0 @@
-name: Lint
-on: [pull_request]
-jobs:
-  lint:
-    runs-on: macOS-11
-    env:
-      DANGER_GITHUB_API_TOKEN: ${{ secrets.DANGER_GITHUB_API_TOKEN }}
-    steps:
-      - uses: actions/checkout@v2
-      - run: bin/download_periphery
-      - run: bundle install
-      - run: bundle exec danger

data/.github/workflows/test.yml

@@ -1,10 +0,0 @@
-name: Test
-on: [push]
-jobs:
-  unit-test:
-    runs-on: macOS-11
-    steps:
-      - uses: actions/checkout@v2
-      - run: bin/download_periphery
-      - run: bundle install
-      - run: bundle exec rake

data/.gitignore

@@ -1,9 +0,0 @@
-.DS_Store
-.idea/
-.yardoc
-/tmp
-bin/lib_InternalSwiftSyntaxParser.dylib
-bin/periphery
-pkg
-project.xcworkspace
-xcuserdata

data/.rspec

@@ -1,3 +0,0 @@
---color
---require spec_helper
---format documentation