[rubygems/rubygems] Skip to load commented out words

rubygems/rubygems@e6a9148ba2

Author: hsbt

lib/bundler/yaml_serializer.rb

@@ -56,6 +56,7 @@ def load(str)
       last_hash = nil
       last_empty_key = nil
       str.split(/\r?\n/) do |line|
+        line = line.split("#", 2).first.strip if line.include?("#")
         if match = HASH_REGEX.match(line)
           indent, key, quote, val = match.captures
           convert_to_backward_compatible_key!(key)

lib/rubygems/yaml_serializer.rb

@@ -56,6 +56,7 @@ def load(str)
       last_hash = nil
       last_empty_key = nil
       str.split(/\r?\n/) do |line|
+        line = line.split("#", 2).first.strip if line.include?("#")
         if match = HASH_REGEX.match(line)
           indent, key, quote, val = match.captures
           convert_to_backward_compatible_key!(key)

spec/bundler/bundler/yaml_serializer_spec.rb

@@ -174,6 +174,21 @@
 
       expect(serializer.load(yaml)).to eq(hash)
     end
+
+    it "skip commented out words" do
+      yaml = <<~YAML
+        ---
+        foo: "bar"
+        buzz: # "foo"
+      YAML
+
+      hash = {
+        "foo" => "bar",
+        "buzz" => {},
+      }
+
+      expect(serializer.load(yaml)).to eq(hash)
+    end
   end
 
   describe "against yaml lib" do

spec/bundler/commands/config_spec.rb

@@ -439,7 +439,7 @@
     it "does not make bundler crash and ignores the configuration" do
       bundle "config list --parseable"
 
-      expect(out).to eq("#mirror.https://rails-assets.org/=http://localhost:9292")
+      expect(out).to be_empty
       expect(err).to be_empty
 
       ruby(<<~RUBY)