Qyriad / Seance

A ritual to channel the unseen
MIT License
11 stars 5 forks source link

option parsing: don't consider argparse defaults over config file values #13

Closed Qyriad closed 2 years ago

Qyriad commented 2 years ago

This was accidentally making prefix specified in an INI config ignored over the argparse default of ''. Oops.

Lunaphied commented 2 years ago

Make sure to document that the INI config is intended to be overridable by the command-line options (or decide to throw up an error for dual definitions).