Support file paths without quoting them

[ajusa]

Hey @euantorano, I saw your message. Here's a minimal example of what I expect to work:

import dotenv

loadEnvFromString("""
FS_ROOT=/home/ajusa/Documents
""")

Error: unhandled exception: (1, 29) Error: symbol expected, but found: /Documents [DotEnvParseError]

If I put the path in quotes, it works as expected (and that's what I'm doing for now). File paths shouldn't require quotes though I would think, since bash doesn't need them either.

[euantorano]

Awesome, thanks for getting back to me. I’ll have a look at this today or tomorrow and see what I can do.

…

On Sat, 29 Jan 2022, at 12:57, Arham Jain wrote: Hey @euantorano <https://github.com/euantorano>, I saw your message. Here's a minimal example of what I expect to work: import dotenv loadEnvFromString(""" FS_ROOT=/home/ajusa/Documents """) `Error: unhandled exception: (1, 29) Error: symbol expected, but found: /Documents [DotEnvParseError] ` If I put the path in quotes, it works as expected (and that's what I'm doing for now). File paths shouldn't require quotes though I would think, since bash doesn't need them either. — Reply to this email directly, view it on GitHub <#3>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFW24OH6MUOJRQ4XCFJQ4DUYPP2VANCNFSM5NC3U6SQ>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[euantorano]

Sorry for the delay, after starting to look I decided to take a different approach and simplify the code a bunch. This should make it a little easier to maintain in the future too, which is nice.

The code is in another branch while I overhaul some stuff and get CI running through GitHub actions. It will be released with a new major version number as the API is changing a bit too to simplify things even further, but you can check it out here: https://github.com/euantorano/dotenv.nim/tree/fix/3-directory-paths