Skip to content
This repository was archived by the owner on Mar 6, 2026. It is now read-only.

chore: add auth team to codeowners file#1088

Merged
clundin25 merged 2 commits intogoogleapis:mainfrom
clundin25:update-codeowners
Jul 26, 2022
Merged

chore: add auth team to codeowners file#1088
clundin25 merged 2 commits intogoogleapis:mainfrom
clundin25:update-codeowners

Conversation

@clundin25
Copy link
Copy Markdown
Contributor

This GitHub team will now be tracked as a codeowner for this repository.

@clundin25 clundin25 requested a review from a team July 22, 2022 17:52
Copy link
Copy Markdown
Contributor

@dandhlee dandhlee left a comment

Choose a reason for hiding this comment

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

We'll also need a Core client library team owner to make admin updates to give write access to the team.

@clundin25
Copy link
Copy Markdown
Contributor Author

We'll also need a Core client library team owner to make admin updates to give write access to the team.

I've pinged @sai-sunder-s

@clundin25 clundin25 force-pushed the update-codeowners branch from 1e4562a to f9abcee Compare July 22, 2022 18:15
@clundin25 clundin25 requested a review from dandhlee July 22, 2022 18:48
@clundin25 clundin25 added the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 22, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 22, 2022
@clundin25 clundin25 added the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 22, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 22, 2022
@clundin25 clundin25 added the automerge Merge the pull request once unit tests and other checks pass. label Jul 22, 2022
@gcf-merge-on-green
Copy link
Copy Markdown
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jul 23, 2022
@clundin25 clundin25 requested a review from TimurSadykov July 25, 2022 15:59
Copy link
Copy Markdown

@TimurSadykov TimurSadykov left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@dandhlee dandhlee left a comment

Choose a reason for hiding this comment

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

LGTM, one minor nit.

@clundin25 clundin25 merged commit 67091d0 into googleapis:main Jul 26, 2022
BigTailWolf pushed a commit to BigTailWolf/google-auth-library-python that referenced this pull request Jul 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants