update OWNERS#1656
Conversation
|
@PatrickXYS can you approve this? For example, I cant approve #1654, unless I am added to this OWNERS. |
|
@thesuperzapper Besides adding you into root OWNER, I'd like to keep more fine-grained ownership to add you into test folder owner and create OWNERs file with yourself listed in common/centraldashboard folder |
e9a6e14 to
da9c9ee
Compare
|
@PatrickXYS I have done this, and also added @kimwnasptd, who is also a lead of the Notebook WG |
|
Thanks @thesuperzapper /lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: PatrickXYS, thesuperzapper The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
As I end up touching so much of this repo (given how many pieces the notebook working group controls), its probably easier if I am added to the root OWNERS files.