KEMBAR78
chore: update codeowners by zhumin8 · Pull Request #1126 · googleapis/google-oauth-java-client · GitHub
Skip to content

Conversation

@zhumin8
Copy link
Contributor

@zhumin8 zhumin8 commented Oct 28, 2024

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:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕️

If you write sample code, please follow the samples format.

@zhumin8 zhumin8 requested review from a team as code owners October 28, 2024 18:47
@product-auto-label product-auto-label bot added the size: xs Pull request size is extra small. label Oct 28, 2024


# The java-samples-reviewers team is the default owner for samples changes
samples/**/*.java @googleapis/java-samples-reviewers
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Error here.

Copy link
Contributor Author

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?

# 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

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.

@product-auto-label product-auto-label bot added size: s Pull request size is small. and removed size: xs Pull request size is extra small. labels Oct 30, 2024
@zhumin8 zhumin8 merged commit c215f7f into main Oct 30, 2024
14 checks passed
@zhumin8 zhumin8 deleted the update-codeowners branch October 30, 2024 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants