Fixed problem with ambiguous dotenv naming

adeharo9 / cpp-dotenv

Loads environment variables from .env files for C++ projects.

BSD 3-Clause "New" or "Revised" License

87 stars 19 forks source link

Fixed problem with ambiguous dotenv naming #22

Closed techbech closed 3 years ago

techbech commented 3 years ago

First off, thank you for the library. I'm happy to see dotenv in cpp :-)

This is a fix for issue #21

I've renamed the dotenv class to dotenvFacade as the purpose of the class seemed to fit the facade pattern. Let me know if you would like the name changed to something else.

adeharo9 commented 3 years ago

Thanks for the suggestion, but it is a bit too verbose for my taste - personal opinion strictly speaking.

Check answer to #21 for more discussion.

techbech commented 3 years ago

Thanks for the suggestion, but it is a bit too verbose for my taste - personal opinion strictly speaking.

Fair :v: