Multiple paths in config: README incorrect? about dotenv HOT 8 CLOSED

Came here to report this same issue, I'm glad someone took a look at it

from dotenv.

released as v16.4.3. big thanks to @thanosd for making dotenv even better for the community!

from dotenv.

it should be a. this is work we wrote up in the README before quite finishing.

a PR would be very welcome. thank you @thanosd !

from dotenv.

I'll spin one out later today. If I also implemented (b) in a more backwards compatible way, would there be interest?

from dotenv.

my mistake. i meant to say b. Fix the code, to perform what is described in the README. the behavior should match the README. the README was written ahead of the behavior in this case.

from dotenv.

OK, I can do that too, but that will take me a day :). Any concerns or changing the implementation? If yes, we could mitigate by adding a flag mergeMultipleFiles, defaulting to false, which would be the current behavior.

from dotenv.

this was a very new implementation (f46336f), so no concerns.

from dotenv.

Took a stab at a PR. Feedback welcome.

from dotenv.