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

[chrome] Import css/[mnop]* triage from TestExpectations #478

Closed

stephenmcgruer wants to merge 1 commit into master from smcgruer/css-css-mnop-triage

Contributor

stephenmcgruer commented Oct 15, 2020

Using the script from #473


          [chrome] Import css/[mnop]* triage from TestExpectations

b205062

Using the script from #473

stephenmcgruer added the do not merge yet label

github-actions bot approved these changes

View reviewed changes

Contributor Author

stephenmcgruer commented Oct 15, 2020

This PR is almost all good cases (yay!), and a few bad cases. List of bad cases, and what dangers they show of importing TestExpectations data:

https://crbug.com/843084 - borderline, this bug is technically valid but is just a "whole bunch of tests fail" meta bug which isn't very useful for understanding the real status.
https://crbug.com/808834 - bug marked Fixed, but tests still fail. Commit message mentions test still failing on Linux, seems weird the bug is Fixed.

stephenmcgruer closed this

KyleJu deleted the smcgruer/css-css-mnop-triage branch

April 23, 2021 18:09

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

Labels

do not merge yet