keep-network / keep-core

The smart contracts and reference client behind the Keep network
https://keep.network
MIT License
118 stars 73 forks source link

Remove deprecation notice for --mainnet flag #3690

Closed nkuba closed 1 year ago

nkuba commented 1 year ago

We planned to remove the --mainnet flag, as it seemed unnecessary, as the client running without any network flag runs for mainnet by default. But from an infrastructure administrator perspective, the flag may be actually useful when running the application on multiple environments. It's much easier to maintain a common template where the administrator will have to change the value of the flag rather than set the flag for some environments and remove it for mainnet.

When running the application for mainnet an administrator will have two possibilities:

When running the application for testnet an administrator will have to add --testnet flag.

dimpar commented 1 year ago

Left just one minor comment. Looks good to me.