divio / divio-cli

Command line interface to the Divio Cloud (formerly aldryn-client)
http://control.divio.com
Other
21 stars 19 forks source link

Fix bash syntax error on divio project pull db #241

Closed ajhalili2006 closed 1 year ago

ajhalili2006 commented 3 years ago

Fixes https://github.com/divio/divio-cli/issues/240

I'm currently testing this on an Ubuntu 20.04.2 Dell computer with Python 3.8 (not sure) and pip 20.x (will update later)

kinkerl commented 2 years ago

Hey, thank you for your contribution! I have a question about the change. I think the quotes around 'db' should collide with the outer quotes ' around the SQL statement. Could you quote the SQL with " instead? In addition, we try to keep everything below 80 chars in line length where possible and this command can be broken down below 80 chars. Do you have a reason for making it all into one line?

Cheers

derlin commented 1 year ago

Closing due to inactivity.