Andrew Klychkov
Andrew Klychkov
@sebasmannem hi, thank you for the issue! Sounds good to me (though I don't work with databases any more but it sounds interesting). I'm happy with any worthwhile improvements that...
I'm not sure if option names `creates` / `removes` can't be confused for some users. Maybe `run_if_true` or something else that would be closer to this context?
+1 to `if_exists`/`if_not_exists` from me too. @hunleyd thanks for the suggestion!
@michaelkebe hello, thanks for reporting this! @tcraxs would you like to take a look?
@caseyandgina thanks for reporting the issue! Would you like to submit a PR? We have the [Quick start guide](https://docs.ansible.com/ansible/devel/community/create_pr_quick_start.html) if needed.
@caseyandgina i reproduced the bug in CI, looking now
After i made the changes from that PR, i get: ``` The full traceback is: File "/tmp/ansible_postgresql_privs_payload_8p69jbhy/ansible_postgresql_privs_payload.zip/ansible_collections/community/postgresql/plugins/modules/postgresql_privs.py", li ne 1166, in main File "/tmp/ansible_postgresql_privs_payload_8p69jbhy/ansible_postgresql_privs_payload.zip/ansible_collections/community/postgresql/plugins/modules/postgresql_privs.py", li ne 843, in manipulate_privs "cannot...
https://github.com/ansible-collections/community.postgresql/pull/335 was created some time ago, when it's tested, we can merge
@JaSafieddine hi, thanks very much for spotting this! Would you like to submit a PR? If you don't have such experience, I'm writing the quick start guide https://github.com/ansible-collections/community.mysql/pull/112. There are...
@JaSafieddine YW! Cool, if any questions, feel free to ask!