paritytech / bench-bot

ISC License
9 stars 19 forks source link

Check for push permissions before running a benchmark #55

Open joao-paulo-parity opened 3 years ago

joao-paulo-parity commented 3 years ago

It's somewhat pointless to run a weights benchmark if the bot is not able to push the generated weights. Some check should be implemented to avoid running the benchmark in that case and, optionally, provide a way of forcing the run through some extra command.

https://github.com/paritytech/bench-bot/issues/54 was proposed but that's supposed to be for recovery in case of unexpected errors, not the default behavior.

Checking for push permissions should be straightforward: