-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Convert CSS font-display test to wpt #6110
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Already reviewed downstream.
Firefox (nightly)Testing web-platform-tests at revision 6871a9d All results1 test ran/css-font-display/font-display.html
|
Sauce (safari)Testing web-platform-tests at revision 6871a9d All results1 test ran/css-font-display/font-display.html
|
Chrome (unstable)Testing web-platform-tests at revision ab26c67 All results1 test ran/css-font-display/font-display.html
|
Sauce (MicrosoftEdge)Testing web-platform-tests at revision ab26c67 All results1 test ran/css-font-display/font-display.html
|
The lint is failing here, I'll update the PR. @jeffcarp @qyearsley, is anyone working on running the link in Chromium's CQ? I think this happens about once a week or so. |
BUG=724932 Review-Url: https://codereview.chromium.org/2895123002 Cr-Commit-Position: refs/heads/master@{#475843}
748adbc
to
94c79e9
Compare
@foolip that task is assigned to me but the current duplicate PR problem is taking priority. Bug for running lints in CQ: https://crbug.com/686927 Hopefully I can resolve the dupes problem soon and work on running the lints. Lmk if that course of action makes/doesn't make sense. |
No, that sounds like the right prioritization to me. |
BUG=724932
Review-Url: https://codereview.chromium.org/2895123002
Cr-Commit-Position: refs/heads/master@{#475843}