Skip to content

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

Merged
andrewrk merged 3 commits into
masterfrom
unwrap-error-switch
Sep 21, 2022
Merged

safety: show error return trace when unwrapping error in switch#12889
andrewrk merged 3 commits into
masterfrom
unwrap-error-switch

Conversation

@Vexu

@VexuVexu commented Sep 17, 2022

Copy link
Copy Markdown
Member

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

@andrewrk

Copy link
Copy Markdown
Member

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

@andrewrkandrewrk self-assigned this Sep 19, 2022
@andrewrkandrewrk removed their assignment Sep 19, 2022
@andrewrk

Copy link
Copy Markdown
Member

The previous CI run passed all checks except Windows.

@andrewrk

Copy link
Copy Markdown
Member

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

@andrewrk

Copy link
Copy Markdown
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 masterSep 21, 2022
@andrewrk
andrewrk deleted the unwrap-error-switch branch September 21, 2022 06:56
Sign up for freeto 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.

2 participants

@Vexu@andrewrk