search

rwnx / pynonymizer

A universal tool for translating sensitive production database dumps into anonymized copies.
https://pypi.org/project/pynonymizer/
MIT License
101 stars 38 forks source link

--help output has duplicate --mysql-cmd-opts #160

Closed simonblake-mp closed 3 months ago

simonblake-mp commented 4 months ago

Describe the bug a section of the output from pynonymizer --help : 

--mysql-cmd-opts                       TEXT                  [mysql] pass additional arguments to the restore process. [env var: PYNONYMIZER_MYSQL_CMD_OPTS]      
--mysql-cmd-opts                       TEXT                  [mysql] pass additional arguments to the dump process. [env var: PYNONYMIZER_MYSQL_DUMP_OPTS]                                                                          │
--mysql-cmd-opts                       TEXT                  [postgres] pass additional arguments to the restore process. [env var: PYNONYMIZER_POSTGRES_CMD_OPTS] 
--mysql-cmd-opts                       TEXT                  [postgres] pass additional arguments to the dump process. [env var: PYNONYMIZER_POSTGRES_DUMP_OPTS]

not sure if those should all be --mysql-cmd-opts down the left hand column?

To Reproduce as above

Expected behavior

--mysql-cmd-opts                       TEXT                  [mysql] pass additional arguments to the restore process. [env var: PYNONYMIZER_MYSQL_CMD_OPTS]      
--mysql-dump-opts                      TEXT                  [mysql] pass additional arguments to the dump process. [env var: PYNONYMIZER_MYSQL_DUMP_OPTS]                                                                          │
--postgres-cmd-opts                    TEXT                  [postgres] pass additional arguments to the restore process. [env var: PYNONYMIZER_POSTGRES_CMD_OPTS] 
--postgres-dump-opts                   TEXT                  [postgres] pass additional arguments to the dump process. [env var: PYNONYMIZER_POSTGRES_DUMP_OPTS]

Additional context

rwnx commented 4 months ago

Looks like an accident with the annotations for the new CLI lib. This is a bug and the behaviour will be restored in v2.2.1. Thanks for your work writing these up!

rwnx commented 3 months ago

These should now be changed to their correct options!