Philip Jägenstedt
Philip Jägenstedt
Yes, we keep metadata in sync with WPT and the Interop 2022 scoring code doesn't follow renames in the past, so a renamed test will behave exactly like a new...
Spelling out the labels in `META.yml` instead of having wpt-pr-bot maintain have that logic would be an improvement IMHO. @stephenmcgruer WDYT?
The labels are used for filtering, unfortunately it's not possible to customize notifications (email) based on labels. Most prominent is probably in the "ongoing work" links in WHATWG specs, see...
A trivial case to show what this looks like in practice: https://github.com/web-platform-tests/wpt/pull/10571#issuecomment-544459487
https://github.com/web-platform-tests/wpt-metadata/pull/804#issuecomment-770963731 has some bits useful for reviewing these PRs: > [bug-titles.txt](https://github.com/web-platform-tests/wpt-metadata/files/5904963/bug-titles.txt) (faux-CSV file; split on the first ',' only). > > Also as a spreadsheet: https://docs.google.com/spreadsheets/d/1AJFl3gLfVFjOXRAir9g2BsesdF3fGp9LJHnDtWQLW-c/edit#gid=0
This is a good idea, and https://help.github.com/en/actions/automating-your-workflow-with-github-actions/events-that-trigger-workflows#webhook-events could probably be used to implement it. It would be interesting to see if the token is read-only or read-write for that event...
https://help.github.com/en/actions/automating-your-workflow-with-github-actions/events-that-trigger-workflows#status-event-status would also be needed if this is done before Taskcluster migrates to GitHub Checks: https://github.com/web-platform-tests/wpt.fyi/issues/1240
It previously would invite people as collaborators to the repo, but I removed that in https://github.com/web-platform-tests/wpt-pr-bot/pull/22. If we downgrade the permissions to write access, are there logs that will reveal...
I have played around with https://github.com/features/actions a bit, enough to convince myself that both the merge_pr_* tagging and manifest upload we do on Travis and the wpt-pr-bot standalone service could...
I spotted in https://github.com/web-platform-tests/wpt-pr-bot/settings/collaboration that @wpt-pr-bot actually has admin access to this very repo. I'm quite sure that's not necessary, so I've removed it.