code_ownership 1.32.2 → 1.32.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 32fde48e3bfda45fef25ea894835a45548aac619699a8c56549000ca373916b5
-  data.tar.gz: e9155c7e859a17a8d31bc0f092822bd161f65a965b356060295be61ee695858b
+  metadata.gz: 3bf52f1bcb70650a93514852fec13bc3e1f4f275643ed712bad37f8439b6ae17
+  data.tar.gz: ca47603f68d95683c4b948318532170a7156651ab24c2f0adcce292975b6076f
 SHA512:
-  metadata.gz: d407ad1c1437c75b7ee9b5df5e22bfe74d90b5e353f62546879fdaf09b7be6d9bbb4c67f70fe0721d55729215377f407f03a83bb094f0fd8d24ad209cbd65957
-  data.tar.gz: dcef3ed24a4af4825a3f8320bcd704885dee0f4f65d7db08515844a67a094dca32f06024275a4d4361908f85f5cbbbdffb27e53c79efab1c384cd1348bb642db
+  metadata.gz: 037e1119a49812006add87fcef7c7a3d6de0505523c36dd7811e0583fbd654ccce450b79a7a6a4b298ef90e511a269ed11d076d823cd4025ceb693e3101c4316
+  data.tar.gz: 5c6ef23bf3554f1531bcebeb6daf49353503a251ee8e3a51ff0223375d3fbb7d4abb1d2f4b0ed99e9368e5279d9c0f623f58906f84b2bc09ed0fb8fecc7d5e33

data/lib/code_ownership/private.rb

@@ -58,10 +58,10 @@ module CodeOwnership
 
     # Returns a string version of the relative path to a Rails constant,
     # or nil if it can't find something
-    sig { params(klass_string: T.nilable(String)).returns(T.nilable(String)) }
-    def self.path_from_klass_string(klass_string)
-      if klass_string
-        path = Object.const_source_location(klass_string)&.first
+    sig { params(klass: T.nilable(T.any(Class, Module))).returns(T.nilable(String)) }
+    def self.path_from_klass(klass)
+      if klass
+        path = Object.const_source_location(klass.to_s)&.first
         (path && Pathname.new(path).relative_path_from(Pathname.pwd).to_s) || nil
       else
         nil

data/lib/code_ownership.rb

@@ -149,23 +149,20 @@ module CodeOwnership
   end
   private_class_method(:backtrace_with_ownership)
 
-  sig { params(klass: T.nilable(T.any(Class, Module, String))).returns(T.nilable(::CodeTeams::Team)) }
+  sig { params(klass: T.nilable(T.any(Class, Module))).returns(T.nilable(::CodeTeams::Team)) }
   def for_class(klass)
     @memoized_values ||= T.let(@memoized_values, T.nilable(T::Hash[String, T.nilable(::CodeTeams::Team)]))
     @memoized_values ||= {}
-
-    klass_string = klass.to_s
-
     # We use key because the memoized value could be `nil`
-    if !@memoized_values.key?(klass_string)
-      path = Private.path_from_klass_string(klass_string)
+    if !@memoized_values.key?(klass.to_s)
+      path = Private.path_from_klass(klass)
       return nil if path.nil?
 
       value_to_memoize = for_file(path)
-      @memoized_values[klass_string] = value_to_memoize
+      @memoized_values[klass.to_s] = value_to_memoize
       value_to_memoize
     else
-      @memoized_values[klass_string]
+      @memoized_values[klass.to_s]
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: code_ownership
 version: !ruby/object:Gem::Version
-  version: 1.32.2
+  version: 1.32.3
 platform: ruby
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-03-10 00:00:00.000000000 Z
+date: 2023-04-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: code_teams