-
Notifications
You must be signed in to change notification settings - Fork 270
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: update codeowners #1126
chore: update codeowners #1126
Conversation
# for handwritten libraries, keep codeowner_team in .repo-metadata.json as owner | ||
**/*.java @googleapis/googleapis-auth | ||
# The @googleapis/cloud-java-team-teamsync is the default owner for changes in this repo | ||
* @googleapis/cloud-java-team-teamsync @googleapis/googleapis-auth | ||
|
||
|
||
# The java-samples-reviewers team is the default owner for samples changes | ||
samples/**/*.java @googleapis/java-samples-reviewers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Error here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for pointing this out. I cross checked with CODEOWNERS in google-auth-library-java here, and suspect the error is this team does not have access to this repo.
However, I don't have settings tab. @westarle @TimurSadykov can you please help verify?
.github/CODEOWNERS
Outdated
# for handwritten libraries, keep codeowner_team in .repo-metadata.json as owner | ||
**/*.java @googleapis/googleapis-auth | ||
# The @googleapis/cloud-java-team-teamsync is the default owner for changes in this repo | ||
* @googleapis/cloud-java-team-teamsync @googleapis/googleapis-auth | ||
|
||
|
||
# The java-samples-reviewers team is the default owner for samples changes | ||
samples/**/*.java @googleapis/java-samples-reviewers | ||
|
||
# Generated snippets should not be owned by samples reviewers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think there are any generated samples in this repo, and I don't think there will be either. It is mostly a concept for GAPIC client libraries. I think we can delete this section.
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
Fixes #<issue_number_goes_here> ☕️
If you write sample code, please follow the samples format.