search

tezos-checker / checker

An in-development "robocoin" system for the Tezos blockchain
24 stars 16 forks source link

ci: Catch errors on the comment workflow #233

Closed utdemir closed 3 years ago

utdemir commented 3 years ago

artifacts.py might fail if the artifacts provided by the job is not of the correct shape. If this happens, it wouldn't be nice for the global comment task to fail. So, this PR wraps it with a try..catch. Unfortunately, as usual, we can not test it on this branch, so we'll see if it fails after merging.

github-actions[bot] commented 3 years ago

Gas costs: No change. Entrypoint sizes: No change. Test coverage: No change.

utdemir commented 3 years ago

By the way, has this happened already?

As far as I know, no. I just had a look at scripts failure cases and noticed this.