Skip to content

Update codeowners.#8261

Merged
busunkim96 merged 2 commits into
masterfrom
codeowners-updates
Jun 7, 2019
Merged

Update codeowners.#8261
busunkim96 merged 2 commits into
masterfrom
codeowners-updates

Conversation

@busunkim96

Copy link
Copy Markdown
Contributor

All guesses were made in good faith. :)

@busunkim96 busunkim96 added the type: process A process-related concern. May include testing, release, or the like. label Jun 6, 2019
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 6, 2019
Comment thread .github/CODEOWNERS Outdated
docs/pubsub @anguillanneuf @plamut


# Bill Prin is the primary maintainer of the Stackdriver libraries.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This person seems to be gone, is it OK to remove this section? @tseaver

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this is safe to remove. This will default to you.

@plamut plamut left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I can approve the Pubsub section, looks good, but can't say much about the other libraries, not familiar with who the people maintaining them are.

Comment thread .github/CODEOWNERS
Comment thread .github/CODEOWNERS Outdated
docs/pubsub @anguillanneuf @plamut


# Bill Prin is the primary maintainer of the Stackdriver libraries.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this is safe to remove. This will default to you.

@busunkim96 busunkim96 merged commit b80d431 into master Jun 7, 2019
@tseaver tseaver deleted the codeowners-updates branch June 28, 2019 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. type: process A process-related concern. May include testing, release, or the like.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants