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

safety: show error return trace when unwrapping error in switch #12889

Merged
merged 3 commits into from Sep 21, 2022

Conversation

Vexu
Copy link
Member

@Vexu Vexu commented Sep 17, 2022

#12807 re-opened on a new branch to modify CI script.

@andrewrk
Copy link
Member

I rebased against latest master branch (2698cb3) now that the CI is fixed.

@andrewrk andrewrk self-assigned this Sep 19, 2022
@andrewrk andrewrk removed their assignment Sep 19, 2022
@andrewrk
Copy link
Member

The previous CI run passed all checks except Windows.

@andrewrk
Copy link
Member

Still failing on Windows. I suggest to run std lib tests in debug mode locally on Windows.

@andrewrk
Copy link
Member

I re-ran the failed Windows test and it passed this time, so we can conclude that the problem affects master branch and was not introduced by this branch.

@andrewrk andrewrk merged commit 902f6db into master Sep 21, 2022
@andrewrk andrewrk deleted the unwrap-error-switch branch September 21, 2022 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants