`loadFile` return value about dotenv-hs HOT 3 CLOSED

Hello @nsaunders Thanks for your question and I'm sorry for this late reply. I went through the commits and I noticed we started with the idea to return unit m () but later we changed it to incorporate a new feature (I assume for convenience) that we ended up removing from this library. IMHO, I prefer to go with the initial approach. In fact, we suggest users to ignore those results (void $ loadFile ...). We also expose parseFile in case someone wants to read the envs from a .env file, so I don't thing we should return the envs from loadFile. I'll change the interface to look as it was before.

Thanks for bringing that to our attention. 😄

from dotenv-hs.

Thanks for your response @CristhianMotoche. For what it's worth, I think the change you are suggesting will remove some ambiguity, and I will plan to follow suit in my own project.

from dotenv-hs.

We've changed the interface of loadFile in v0.10.0.0. Once again, thanks for reporting this. 👍

from dotenv-hs.