euantorano / dotenv.nim Goto Github PK

When attempting to utilize this package with the latest nim version (0.17.0) I was getting the following error while running the test suite:

$ nim c -r tests/main.nim 
/Users/yuliy/code/dotenv.nim/dotenv.nim(40, 38) template/generic instantiation from here
lib/system.nim(2656, 7) Error: can raise an unlisted exception: ref AssertionError

Which I believe stems from the addition of a doAssert in the core os module (found here).

I believe that this change in my fork should resolve the issue and maintain backwards compatibility with previous versions, but I am fairly new to nim so any feedback is appreciated!

Do you think that the examples should add import std/os, so that someone will know to import std/os otherwise it will not work? Just a suggestion

It happens in v2.0.0

.env

HOGE=true # ture or false

nim

import os, dotenv

load(getCurrentDir(), ".env")
getEnv("HOGE") == "true # ture or false"

but expected getEnv(HOGE) is "ture"

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.