-
Notifications
You must be signed in to change notification settings - Fork 48
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
Update s* from monorail results #829
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.
Did some spot checking.
@@ -0,0 +1,12 @@ | |||
links: | |||
- product: chrome | |||
url: https://crbug.com/1056055 |
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.
This is a success case, WPT import bug got turned into something more specific.
@stephenmcgruer oops this conflicts now, sorry for slow review. |
See #798 for the methodology
a31cb76
to
55d8ec6
Compare
And it even conflicts with work someone had to do manually :P. I've pushed the branch, one interesting case is svg/path/closepath/META.yml where the script came up with a different bug than the human. I think the script is actually correct here, so I'm keeping the script's bug. |
See #798 for the methodology