Skip to content

Comments

Fix CODEOWNERS#275

Merged
c-parsons merged 1 commit intomasterfrom
fix_owners
Sep 15, 2020
Merged

Fix CODEOWNERS#275
c-parsons merged 1 commit intomasterfrom
fix_owners

Conversation

@achew22
Copy link
Member

@achew22 achew22 commented Sep 14, 2020

Previously permissions were granted to edit any file that existed in a directory
titled gazelle. Now grants have been given recursively.

Sorry for the inconsistency with Google3 OWNERS files.

@aiuto & @c-parsons . please review this very carefully since I have no prior
experience with CODEOWNERS. I can say, however, that neither @jayconrod
nor I presently have the ability to submit code in the /gazelle dir, which is
why I'm messing with this.

Previously permissions were granted to edit any file that existed in a directory
titled `gazelle`. Now grants have been given recursively.

Sorry for the inconsistency with Google3 `OWNERS` files.
@achew22
Copy link
Member Author

achew22 commented Sep 14, 2020

I tried to send this to a subset of the root owners, but that doesn't appear to be possible. Nor does it seem possible for me to remove you. Sorry to broadcast with such a high replication factor!

@jin jin removed request for jin and laurentlb September 14, 2020 04:40
@achew22
Copy link
Member Author

achew22 commented Sep 15, 2020

Friendly ping

@c-parsons c-parsons merged commit 2a89db4 into master Sep 15, 2020
@achew22 achew22 deleted the fix_owners branch September 15, 2020 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants