Skip to content
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.

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

Fix not reported when log from not main thread #21

Merged
merged 4 commits into from
Nov 14, 2023

Conversation

Kuniwak
Copy link
Member

@Kuniwak Kuniwak commented Nov 9, 2023

Reproducing steps

  1. Checkout b573d0a
  2. Stay keep state not playing
  3. Open the scene "Error"
  4. Open an inspector for "AutopilotSettingsForFacingImmediatelyError"
  5. Click Run

Expected Behaviour

Stop playing and report the error to Slack.

Actual Behaviour

Still playing and did not report to Slack.


Contribution License Agreement

@Kuniwak Kuniwak force-pushed the stop-when-error-immediately-thrown branch from f27dc9e to 3e9deab Compare November 9, 2023 20:10
@Kuniwak Kuniwak self-assigned this Nov 9, 2023
@Kuniwak Kuniwak force-pushed the stop-when-error-immediately-thrown branch 3 times, most recently from 049f935 to 9af66be Compare November 9, 2023 20:29
Reproducing steps:
1. Stay keep state not playing
2. Open the scene "Error"
3. Open an inspector for "AutopilotSettingsForFacingImmediatelyError"
4. Click Run

Expected Behaviour:
Stop playing and report the error to Slack.

Actual Behaviour:
Still playing and did not report to Slack.
@Kuniwak Kuniwak force-pushed the catch-from-not-main-thread branch from 4c0c018 to b573d0a Compare November 9, 2023 20:30
@Kuniwak Kuniwak added the bug Something isn't working label Nov 9, 2023
@Kuniwak Kuniwak marked this pull request as ready for review November 9, 2023 20:51
@Kuniwak Kuniwak requested a review from a team as a code owner November 9, 2023 20:51
asurato
asurato previously approved these changes Nov 14, 2023
Base automatically changed from stop-when-error-immediately-thrown to master November 14, 2023 05:19
@Kuniwak Kuniwak dismissed asurato’s stale review November 14, 2023 05:19

The base branch was changed.

@Kuniwak Kuniwak merged commit 4d493bb into master Nov 14, 2023
1 check passed
@Kuniwak Kuniwak deleted the catch-from-not-main-thread branch November 14, 2023 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants