New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Handle errors in pre-play hook #525

Merged

goto-bus-stop merged 3 commits into default from pre-play-error

Jun 22, 2022

Member

goto-bus-stop commented Jun 22, 2022

An error in a source pre-play hook could cause the entire process to crash. Now we log the error but keep going as if the hook didn't exist.

goto-bus-stop added 2 commits

June 22, 2022 22:40


          Handle errors in pre-play hook

876d21e


          fix type

8a53de3

coveralls commented Jun 22, 2022 •

edited

Loading

Coverage decreased (-0.04%) to 78.316% when pulling 84fdb47 on pre-play-error into 564713e on default.


          Merge branch 'default' into pre-play-error

84fdb47

goto-bus-stop enabled auto-merge (squash)

June 22, 2022 20:49

goto-bus-stop merged commit eedfb9b into default

goto-bus-stop deleted the pre-play-error branch

June 22, 2022 20:52

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet