Rethink writing of double quotes around value to in flatten_and_write

[theskumar]

Rethink writing of double quotes around value to in flatten_and_write

[tino]

Yeah, they do make sense. The problem was that the tool loading the environment variables (not with python-dotenv) was tripping upon them. But I guess I better fix that tool :)

[hugochinchilla]

I use this library to mantain my dotenv files across a number of projects, but I have a problem with Docker and it's that for Docker an envfile gets parsed as if you split the lines by the equal sign, everything before the equal is the variable name, the rest is the value, the problem is that you end up with quotes in your values which breaks some applications.

I guess changing the behavior by default could be a breaking change, maybe a good compromise is to support a flag to set the behavior:

 --quote=always (current behavior)
 --quote=never (never add quotes)
 --quote=auto (only add quotes for values containing spaces)

I would add the "auto" behavior to support sourcing dotenv files from bash, i use this a lot while debugging, the problem with unquoted strings containing whitespace is that bash thinks you are trying to run a command.

[theskumar]

Just released 0.4.0 with this functionality. 🎉

@hugochinchilla are you able to install the new version and confirm it works for you. Thanks.

[hugochinchilla]

Works amazingly, thank you for merging so fast.

[theskumar]

Released 0.6.0 containing this change is on PYPI. Yay!

[hugochinchilla]

@theskumar I think you wanted to post your last comment on PR #30 ;)

Thanks for merging :)