Skip to content
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: add dougthor42 as one of the gazelle owners #2427

Merged
merged 1 commit into from
Nov 21, 2024
Merged

Conversation

aignas
Copy link
Collaborator

@aignas aignas commented Nov 19, 2024

Related #2231

@Geethree, @ewianda, let me know if you would also like to start reviewing PRs for gazelle. If you would otherwise like to do adhoc PR reviews, feel free to chime in whenever you like.

Related #2231

@Geethree, @ewianda, let me know if you would also like to start reviewing PRs for `gazelle`. If you would otherwise like to do adhoc PR reviews, feel free to chime in whenever you like.
@aignas
Copy link
Collaborator Author

aignas commented Nov 19, 2024

FYI @dougthor42

@Geethree
Copy link

I’ll start reviewing PRs :)

Copy link

@Geethree Geethree 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
Contributor

@dougthor42 dougthor42 left a comment

Choose a reason for hiding this comment

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

Happy to help!

I assume the write access error will be resolved later?

@ewianda
Copy link
Contributor

ewianda commented Nov 19, 2024

Happy to start reviewing PRs as well

@aignas
Copy link
Collaborator Author

aignas commented Nov 20, 2024

@dougthor42, yeah the write access for you would have to be resolved by someone within Google, maybe @rickeylev can help.

@rickeylev rickeylev added this pull request to the merge queue Nov 21, 2024
Merged via the queue into main with commit 1aee4bf Nov 21, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants