Cardinal-Cryptography / aleph-node-runner

Apache License 2.0
12 stars 9 forks source link

Allow manually overriding the version to run. Remove session key checks. #54

Closed piotrMocz closed 1 year ago

piotrMocz commented 1 year ago

The runner won't try to figure out the version by calling the System::version extrinsic. Instead, it will, by default, use curl to fetch the version from this repo's main branch. If that fails, it will display an error message and exit. It is now possible to override the version with the --version flag but the runner will prompt you whether you are sure you know what you're doing.

Additionally, the session keys checks have been removed: they provided a questionable user experience and relied on cliain, introducing one more point of failure.

Bonus: way nicer output. WITH COLORS. Bonus 2: cleaner code with less ifs :)