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

Revert "[wasm] Revert to using latest chrome for testing" #83234

Closed
wants to merge 1 commit into from

Conversation

thaystg
Copy link
Member

@thaystg thaystg commented Mar 9, 2023

It looks like the failures on CI are still happening.

Reverts #83150

@thaystg thaystg requested a review from radical March 9, 2023 22:16
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ghost ghost assigned thaystg Mar 9, 2023
@thaystg thaystg requested a review from lewing March 9, 2023 22:18
@radical
Copy link
Member

radical commented Mar 9, 2023

Were these failures hit on PRs?

@lewing
Copy link
Member

lewing commented Mar 9, 2023

Were these failures hit on PRs?

lots of timeouts

@radical
Copy link
Member

radical commented Mar 9, 2023

Were these failures hit on PRs?

lots of timeouts

On debugger tests?

@thaystg
Copy link
Member Author

thaystg commented Mar 9, 2023

#83208

@lewing
Copy link
Member

lewing commented Mar 10, 2023

and of course it timed out ;)

@thaystg
Copy link
Member Author

thaystg commented Mar 10, 2023

I will investigate it tomorrow my morning! :)

@radical
Copy link
Member

radical commented Mar 14, 2023

@thaystg should this be merged?

@thaystg
Copy link
Member Author

thaystg commented Mar 15, 2023

@thaystg should this be merged?

No

@thaystg thaystg closed this Mar 15, 2023
@jkotas jkotas deleted the revert-83150-chrome-latest branch March 25, 2023 04:38
@ghost ghost locked as resolved and limited conversation to collaborators Apr 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants