serokell / xrefcheck

Check cross-references in repository documents
Mozilla Public License 2.0
54 stars 3 forks source link

Revise behaviour of `dump-config` command #254

Closed Martoon-00 closed 1 year ago

Martoon-00 commented 1 year ago

Clarification and motivation

Currently this command immediately writes to a file, while an intuition would say that it should just print config to the console. (and actually, it was me who implemented it this way :sweat:). But okay, this has some advantages.

But this command can also overwrite the previously created config. I.e. I could create config and tune it up, then accidentally call xrefcheck dump-config again (e.g. by getting this command from history) and all my changes will be lost. This is too bad.

Acceptance criteria