[webkit-dev] Add CODEOWNERS to WebKit

[Jonathan Bedard]

> On Jun 2, 2022, at 2:53 PM, Alexey Proskuryakov <ap at webkit.org> wrote:
> 
> 
> It seems like we still need to host watchlist CC service though, to support patch workflow - and we'll have separate lists in GitHub and in Bugzilla, unless watchlists are reimplemented.

We would still need to host the current watchlist service, yes. Although we wouldn’t have to teach this watchlist how to interact with GitHub. It would also be possible (and not too hard) to teach our watchlist service to read the CODEOWNERS file, at least in cases where the individual being CCed wasn’t a group.

> 
> I believe that while there are stale watchlists entries, newer additions are valid.
> 
> - Alexey
> 
>> 2 июня 2022 г., в 2:19 PM, Jonathan Bedard via webkit-dev <webkit-dev at lists.webkit.org <mailto:webkit-dev at lists.webkit.org>> написал(а):
>> 
>> Porting is possible, but that would be something that would happen, at the earliest, a few weeks from now. I does seem to me that the watchlist is out of date, so a forced-cleanup doesn’t seem like the worst thing.
>> 
>> Jonathan
>> 
>>> On Jun 2, 2022, at 2:21 PM, Chris Dumez <cdumez at apple.com <mailto:cdumez at apple.com>> wrote:
>>> 
>>> I’m kind of ambivalent/neutral about this. One question though:
>>> 
>>> When adopting CODEOWNERS, will our existing watchlists get ported, or will each contributor have to modify CODEOWNERS themselves to match whatever was in the watchlists for them?
>>> 
>>> Thanks,
>>> Chris.
>>> 
>>>> On Jun 2, 2022, at 1:12 PM, Jonathan Bedard via webkit-dev <webkit-dev at lists.webkit.org <mailto:webkit-dev at lists.webkit.org>> wrote:
>>>> 
>>>> Hey folks,
>>>> 
>>>> Yusuke is interested in adding the CODEOWNERS file to the WebKit project (see https://github.com/WebKit/WebKit/pull/1137 <https://github.com/WebKit/WebKit/pull/1137>). There have been some objections to this, the two ones I’m aware of:
>>>> - WebKit doesn’t assign “ownership” to pieces of code, so the name is unfortunate
>>>> - The last match “wins”, so if you subscribed to a generic directory early in the file, more specific matches would override that subscription
>>>> 
>>>> Despite these downsides, I think adding the CODEOWNERS file is good for the project for a few reasons:
>>>> - It’s a standard natively supported by GitHub, BitBucket and GitLab and will be familiar to developers
>>>> - File format is more simple than existing watchlist
>>>> - Support for groups and individuals
>>>> - No need for WebKit to host a service (other implementations of auto-CCing would require this)
>>>> 
>>>> I intend to work with Yusuke to land https://github.com/WebKit/WebKit/pull/1137 <https://github.com/WebKit/WebKit/pull/1137> and start adopting CODEOWNERS on Monday absent objections that folks think overshadow the benefits of the CODEOWNERS file.
>>>> 
>>>> Jonathan
>>>> _______________________________________________
>>>> webkit-dev mailing list
>>>> webkit-dev at lists.webkit.org <mailto:webkit-dev at lists.webkit.org>
>>>> https://lists.webkit.org/mailman/listinfo/webkit-dev <https://lists.webkit.org/mailman/listinfo/webkit-dev>
>>> 
>> 
>> _______________________________________________
>> webkit-dev mailing list
>> webkit-dev at lists.webkit.org <mailto:webkit-dev at lists.webkit.org>
>> https://lists.webkit.org/mailman/listinfo/webkit-dev
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20220606/e3f2c532/attachment.htm>