web-platform-tests / rfcs

web-platform-tests RFCs
74 stars 63 forks source link

RFC #163: Add web_feature metadata file to web-platform-tests #163

Closed jcscottiii closed 8 months ago

jcscottiii commented 10 months ago

Replacement of https://github.com/web-platform-tests/rfcs/pull/157 that uses the feedback from the last WPT meeting.

Rendered

foolip commented 10 months ago

https://github.com/web-platform-dx/web-features has been renamed to match the name of the NPM package, so I suggest renaming this to web_features or features.

jcscottiii commented 10 months ago

Thanks for pointing that out @foolip ! I have renamed things.

foolip commented 9 months ago

@jcscottiii did you forget to push changes? There's a lot of "feature-set" in the text still.

jcscottiii commented 9 months ago

@foolip that's quite embarrassing of me 😅. Thanks for catching that.

I fixed that.

jcscottiii commented 9 months ago

@jgraham I added the lint suggestion that you mentioned during the https://github.com/web-platform-tests/wpt-notes/blob/master/minutes/2023-09-05.md

foolip commented 9 months ago

@Ms2ger are you OK with the wpt.fyi integration being out of scope? You have a "requested changes" review if you'd like to give it another spin :)

foolip commented 9 months ago

@jcscottiii now that the file can list multiple features, can you rename it from WEB_FEATURE.yml to plural WEB_FEATURES.yml? This aligns with the package name web-features too :)

jcscottiii commented 9 months ago

@foolip Thanks for the suggestion. Done!

jcscottiii commented 8 months ago

@jgraham any thoughts on the latest version

foolip commented 8 months ago

A first WEB_FEATURES.yml file was added in https://github.com/web-platform-tests/wpt/pull/42709. Thanks @jcscottiii for driving this RFC!