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

[ci] update codeowners #6542

Merged
merged 1 commit into from
Jul 16, 2024
Merged

[ci] update codeowners #6542

merged 1 commit into from
Jul 16, 2024

Conversation

jameslamb
Copy link
Collaborator

Proposes adding @StrikerRUS back to CODEOWNERS, to be automatically added as a reviewer on PRs.

@StrikerRUS since #5998, we've stopped having specialization in the CODEOWNERS file (no more "these people are R reviewers", "these people are Python reviewers", etc.) and switched to just having everyone added to every PR by default. That more closely matched the current state of the project and abilities/interests of the reviewers.

If you don't want to be added as a reviewer on every PR by default let me know, and we can close this.

@StrikerRUS
Copy link
Collaborator

I would be honored!

To be honest, I'm afraid that I won't be active as much as I was in the past 😢
But I'll do my best to help with this project!

@jameslamb
Copy link
Collaborator Author

Thanks very much!

Totally understand, we'll be happy for any time you're able to spend helping out here.

@StrikerRUS
Copy link
Collaborator

As always only kind words from you, James! Thanks a lot! 🤗

@guolinke
Copy link
Collaborator

@StrikerRUS very excited to have you back!

@StrikerRUS
Copy link
Collaborator

@guolinke Hi! 👋
Glad to to see you! It's a great pleasure to work with you.

@jameslamb jameslamb merged commit 3d02662 into master Jul 16, 2024
45 checks passed
@jameslamb jameslamb deleted the update-codeowners branch July 16, 2024 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants