romanz / electrs

An efficient re-implementation of Electrum Server in Rust
MIT License
1.02k stars 373 forks source link

deprecate unused config options `daemon_dir` and `timestamp` #1052

Open antonilol opened 2 weeks ago

antonilol commented 2 weeks ago

closes #1044

Feel free to edit any of the messages by directly committing (but review is still fine)

Is std::process::exit(1); desirable? This forces users to remove the flag with a nice message, in future releases they will get an error from the arg parser. When not exiting people might (and will) miss it, how many people look at the logs when booting a program? (thinking about one-click install full node distros for example)