Make CodeManifest.load_manifes publict (#5)

* Make CodeManifest.load_manifes publict

* Bump

* Fix

* Review tweaks

Author: tubaxenor

Gemfile.lock

@@ -1,12 +1,15 @@
 PATH
   remote: .
   specs:
-    code_manifest (1.6.1)
+    code_manifest (1.7.0)
+      psych (>= 4.0.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
     diff-lcs (1.5.0)
+    psych (5.1.2)
+      stringio
     rake (13.0.6)
     rspec (3.11.0)
       rspec-core (~> 3.11.0)
@@ -21,6 +24,7 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.11.0)
     rspec-support (3.11.1)
+    stringio (3.1.0)
 
 PLATFORMS
   arm64-darwin-21

code_manifest.gemspec

@@ -21,7 +21,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   # Uncomment to register a new dependency of your gem
-  # spec.add_dependency "example-gem", "~> 1.0"
+  spec.add_dependency "psych", ">= 4.0.0"
 
   spec.add_development_dependency 'rspec', '~> 3.0'
 

lib/code_manifest.rb

@@ -21,20 +21,20 @@ def root(start_path: Dir.pwd, reset: false)
       @root ||= find_root(start_path)
     end
 
-    private
+    def load_file(manifest_file)
+      YAML.load_file(manifest_file, aliases: true).each_with_object({}) do |(name, patterns), collection|
+        next unless name.match?(KEY_PATTERN)
 
-    def manifests
-      @manifests ||= begin
-        manifest_file = root.join(MANIFEST_FILE)
+        raise ArgumentError, "#{name} defined multiple times in #{manifest_file}" if collection.key?(name)
 
-        load_manifest(manifest_file).each_with_object({}) do |(name, patterns), collection|
-          next unless name.match?(KEY_PATTERN)
+        collection[name] = Manifest.new(patterns.flatten)
+      end
+    end
 
-          raise ArgumentError, "#{name} defined multiple times in #{MANIFEST_FILE}" if collection.key?(name)
+    private
 
-          collection[name] = Manifest.new(patterns.flatten)
-        end
-      end
+    def manifests
+      @manifests ||= load_file(root.join(MANIFEST_FILE))
     end
 
     def find_root(path)
@@ -44,12 +44,5 @@ def find_root(path)
 
       raise "#{MANIFEST_FILE} was not found in your project directory, please check README for instructions."
     end
-
-    # https://stackoverflow.com/a/71192990
-    def load_manifest(file)
-      YAML.load_file(file, aliases: true)
-    rescue ArgumentError
-      YAML.load_file(file)
-    end
   end
 end

lib/code_manifest/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CodeManifest
-  VERSION = '1.6.1'
+  VERSION = '1.7.0'
 end