search

motdotla / dotenv

Loads environment variables from .env for nodejs projects.
https://www.dotenvx.com
BSD 2-Clause "Simplified" License
19.18k stars 859 forks source link

Fix: Redundant code removal. #761

Closed joyson-madtha0 closed 1 year ago

joyson-madtha0 commented 1 year ago

I was stuck integrating dotenv with vitest on nextjs app. While doing reverse engineering on dotenv to figure out what I am doing wrong, I stumbled on to this piece of code. Hope this fix will help a little.

Anyway, I did integrated dotenv with vitest. This change is not a blocker for me...

motdotla commented 1 year ago

This has some purpose with different import vs require approaches. Going to leave as is. Thank you for the PR though.