[webkit-dev] Add CODEOWNERS to WebKit

[Jonathan Bedard]

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> 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> 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
>> https://lists.webkit.org/mailman/listinfo/webkit-dev
> 

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