You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
autonick immediately runs and changes nicknames of all un-nicked dwarves when run with any arguments. This behavior is inconsistent with all other dfhack scripts and utilities, which generally accept "help" or "-help" to give detail about what the command does and how to use it. This leads to new users finding themselves with all their dwarves renamed, with no confirmation, and no way to change them back automatically.
Suggested remediations:
Add a confirmation dialog, and a flag to suppress it if someone wants to script autonick
Listen for "help" and "-help" and give instructions if they are found
Throw an error if autonick does not recognize an argument
The text was updated successfully, but these errors were encountered:
after a cursory look at some other scripts to see what the best practices might be, a majority seem to ignore -h / -help arguments. drain-aquifer runs destructively without complaint while fillneeds errors with a stack trace, for example. neither display help. i'll continue working on autonick, but it might be worth going through the list of scripts more thoroughly and documenting which ones need the same treatment.
It handles "-h", "--help", and "help". It might work for "-help" too since that's the same as "-h -e -l -p" and "-h" is the first param that is processed.
autonick immediately runs and changes nicknames of all un-nicked dwarves when run with any arguments. This behavior is inconsistent with all other dfhack scripts and utilities, which generally accept "help" or "-help" to give detail about what the command does and how to use it. This leads to new users finding themselves with all their dwarves renamed, with no confirmation, and no way to change them back automatically.
Suggested remediations:
The text was updated successfully, but these errors were encountered: