search

web-platform-tests / wpt

Test suites for Web platform specs — including WHATWG, W3C, and others
https://web-platform-tests.org/
Other
4.91k stars 3.06k forks source link

Document the use of labels #8286

Open foolip opened 6 years ago

foolip commented 6 years ago

I can't find any documentation of the labels in use in this repo. I'd like to write that documentation, and based on that work out some triage process for groups of labels.

@tobie, where is the code that automatically creates labels for each directory?

foolip commented 6 years ago

FWIW, these are the directory names in / and /css/ that don't have a label:

Seems about right, but suggests there's some special handling for XHR.

And these are the labels that don't correspond to a directory:

Mostly reasonable, but some stuff in need of cleanup, like WARNING and tmp. (I'll clean a bit now.)

tobie commented 6 years ago

https://github.com/tobie/ganesh/blob/master/lib/metadata/raw-labels.js

and

https://github.com/tobie/ganesh/blob/master/lib/metadata/labels.js

gsnedders commented 6 years ago

@foolip note if someone creates a PR with a new directory a label gets created for it, so it's quite likely we'll have labels that don't correspond to a directory

foolip commented 6 years ago

Yep, that's a thing I'd like to document, that if you push a PR with a "test" directory, a test label will be created. People probably don't expect that :)