ruby-lsp-reek 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8dc0f0d5264c4cb55e08c39e863d49113fc206876b594c37f1c5b495a04c18fc
-  data.tar.gz: a5cacca4c13f0008b4ef9868ee9a4c557e296a0f64ebf9ac8951ad3771fe9875
+  metadata.gz: 2a825d18e3f5825299c0d0e6af7682e3a1f8aa2bc2a030a96a807eaa12aced63
+  data.tar.gz: c001a2068adb09a8c9610130da10d4ce9bdbb4e534c587b28a13ba53ad1fe42b
 SHA512:
-  metadata.gz: 725b73500121ff791293945ff1fc6a26508abc50d77715a127966fd0de73071fbd51f667643579f5499249cbdfc1f787d8495621b9d56e1e44037fbd1bd045f4
-  data.tar.gz: 595b0d95becf70e30584691601dec34d3a84aab1c390cbeed5ae4ff50c8060d96d0d0e0be62e8ef530ed0088c75341212e951e708e4fac41a5a58fe3ebc7e8b2
+  metadata.gz: 0cec537f3c7d5d39842c75a6715b54ae39999784bedc3b331b2f98effc248847b4a8daf534e193ae055a68b0aeef3b3f5af17b5aae7610510bb095b578dfbf3f
+  data.tar.gz: 3b8f36ccfc2455b2af3c6bb548c67b95e1ce95a72f98ed55138ec5efebf237fe2d2453822f09eef1bb902490794f2e6dcc1efb67fa793f083dea65e725ac5040

data/README.md

@@ -14,10 +14,9 @@ And then execute:
 
     $ bundle
 
-Note: `ruby-lsp` does not seem to recognize this addon when installed outside
-of bundler, but YMMV:
-
-    $ gem install ruby-lsp-reek
+Note: Using globally installed linters in Ruby LSP is [not
+supported](https://github.com/Shopify/ruby-lsp/blob/main/vscode/README.md#formatting),
+they must in your Gemfile or gemspec.
 
 ## Usage
 

data/lib/ruby_lsp/reek/addon.rb

@@ -68,7 +68,7 @@ module RubyLsp
       def workspace_did_change_watched_files(changes)
         return unless changes.any? { |change| change[:uri].end_with?(".reek.yml") }
 
-        runner.init!
+        @runner = Runner.new
         warn "Re-initialized Reek Ruby LSP addon v#{::RubyLsp::Reek::VERSION} due to .reek.yml file change"
       end
 

data/lib/ruby_lsp/reek/runner.rb

@@ -24,9 +24,10 @@ module RubyLsp
       # @param uri [String] The URI of the document to run diagnostics on.
       # @param document [RubyLsp::Interface::TextDocumentItem] The document to run diagnostics on.
       def run_diagnostic(uri, document)
-        return [] if config.path_excluded?(Pathname.new(uri.path))
+        path = Pathname.new(uri.path)
+        return [] if path_excluded?(path)
 
-        examiner = ::Reek::Examiner.new(document.source, configuration: config)
+        examiner = ::Reek::Examiner.new(path, configuration: config)
         examiner.smells.map { |smell| warning_to_diagnostic(smell) }
       end
 
@@ -56,6 +57,14 @@ module RubyLsp
           message: warning.message
         )
       end
+
+      def path_excluded?(path)
+        path.ascend do |ascendant|
+          break true if config.path_excluded?(ascendant)
+
+          false
+        end
+      end
     end
   end
 end

data/lib/ruby_lsp/reek/version.rb

@@ -2,6 +2,6 @@
 
 module RubyLsp
   module Reek
-    VERSION = "0.2.0"
+    VERSION = "0.3.0"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ruby-lsp-reek
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Iain Gray
@@ -194,5 +194,5 @@ requirements: []
 rubygems_version: 3.5.11
 signing_key:
 specification_version: 4
-summary: Ruby LSP Reek
+summary: Reek addon for Ruby-LSP
 test_files: []