Merge pull request #291 from sloria/skip-nonenvar-lines

Skip over lines that don't set an envvars
bkeepers · Apr 13, 2018 · b44474f · b44474f
2 parents 22cfc3f + 697865b
commit b44474f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/lib/dotenv/parser.rb b/lib/dotenv/parser.rb
@@ -59,8 +59,6 @@ def parse_line(line)
  if variable_not_set?(line)
  raise FormatError, "Line #{line.inspect} has an unset variable"
  end
- elsif line !~ /\A\s*(?:#.*)?\z/ # not comment or blank line
- raise FormatError, "Line #{line.inspect} doesn't match format"
  end
  end
 

diff --git a/spec/dotenv/parser_spec.rb b/spec/dotenv/parser_spec.rb
@@ -119,8 +119,8 @@ def env(string)
  expect(env("foo=bar ")).to eql("foo" => "bar") # not 'bar '
  end
 
- it "throws an error if line format is incorrect" do
- expect { env("lol$wut") }.to raise_error(Dotenv::FormatError)
+ it "ignores lines that are not variable assignments" do
+ expect(env("lol$wut")).to eql({})
  end
 
  it "ignores empty lines" do