-
Notifications
You must be signed in to change notification settings - Fork 256
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
Proposals for new maintainers and reviewers #1078
Comments
I agree we need more maintainer who can add /approve. |
I'd suggest @tobiasgiese and @mdbooth |
I'd also be happy to help :-) I also plan to get more time to do upstream work for next year 👍 |
I also +1 @mdbooth as maintainer given the amount and quality of activity on the project over the last year. For my part, I'd be happy to take on doing more reviews more regularly in CAPO as a reviewer. |
+1 for @tobiasgiese and @mdbooth @tobiasgiese worked on issues like Provide documentation website (like CAPA/CAPZ)(#783), and Change default branch to "main"(#855) as practical maintainer. |
Just to clarify, I also worked on other issues except for documentation and branch renaming 😅 (xref) And thanks for the suggestion 🙂 |
I would also be happy to help out as a maintainer, although I will say that there are changes that I wouldn't yet feel confident to approve myself. It's good to have a broad team! |
And I'm also happy to help as a maintainer, for sure! I forgot to mention that 😃 |
As we've got a fair bit going on at the moment, I wonder if we should consider adding to the reviewer pool. Purely to start the discussion, how would we feel about the following additions?
Maintainer
Reviewer
This is a straw man from the top of my head! My apologies if I missed anybody obvious 😬
/cc @jichenjc @hidekazuna
The text was updated successfully, but these errors were encountered: