[WPT common] Full support of Chromium OWNERS syntax |
|||||
Issue descriptionCurrently, our DirectoryOwnersExtractor is a homegrown simplified version of OWNERS parser, only supporting the minimal syntax of one email per line (implemented in issue 713987 ). There have been use cases where full syntax support (the "file://" directive in this example) is desired: https://chromium-review.googlesource.com/c/chromium/src/+/645528/2/third_party/WebKit/LayoutTests/external/wpt/accelerometer/OWNERS#1 The best approach would be to investigate how to invoke depot_tools (owners.py) to avoid reinventing the wheel.
,
Sep 4 2017
In https://chromium-review.googlesource.com/c/chromium/src/+/645528/2#message-adc7b287dc2c61f9b5c3751e11f3dc6717017c9c dpranke@ says "For what it's worth, I've posted a CL to add a `git-cl owners --batch` command that can allow you to non-interactively get a covering list of owners/reviewers for your CL. That would hopefully be useful here."
,
Sep 4 2017
I hope this is not too cavalier, but I personally haven't found the notification emails terribly useful. That's because unless a directory is already in very good shape, reacting to each additional failure as it happens, even if only by filing a bug, isn't the best way to prioritize work in that area. Rather, one would want some tooling to understand all the current failures, and whether there are bugs filed for each. So, I'll go ahead and land https://chromium-review.googlesource.com/c/chromium/src/+/645528 without this being fixed, since the directories were previously unowned.
,
Sep 6 2017
Started a broader discussion on the notification mechanism on blink-dev: https://groups.google.com/a/chromium.org/d/topic/blink-dev/L4-7ZIfMK3Q/discussion
,
Sep 14 2017
Well it's not exactly "blocking", but I'm linking this issue to bug 765334 as that's the umbrella bug for the new notification mechanism.
,
Nov 29 2017
,
Nov 30
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 6
"git-cl owners" has had a --batch option for more than a year now; I guess this would still be useful to have. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by qyears...@chromium.org
, Sep 1 2017