-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Idea for new lint check: missing META.yml for new top-level directory #10552
Comments
@gsnedders says
|
You are right. We don't sync OWNERS files. If we want to add a wpt lint for this, we may need a command-line flag to turn it off, which should be useful to us and perhaps others as well. |
That may solve itself when we rename OWNERS to NOTIFY. |
OWNERS have been renamed to META.yml and those are imported into Blink, so the lint problem is gone. |
@zcorpan is this something you're planning to tackle? It looks like I end up adding them more often that I'd like: |
I can try to fit this in as part of the review velocity work. |
Related to #5915
It'd be nice if new top-level directories have OWNERS files from the start when they are created (or subdirectory of css/).
cc @gsnedders
The text was updated successfully, but these errors were encountered: