-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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
boards: we: correct board name #87285
Conversation
correct board name in documentation Signed-off-by: Matthias Hauser <matthias.hauser@we-online.de>
@jhedberg hey, how are reviewers assigned to my PR? This one is really trivial. |
This has to be done manually, since there's no entry for |
Thank you @jhedberg , I did not know that. I believe this year we are able to join the group of maintainers as well. Just for curiosity: How can we mark a certain account as maintainer? Add it to the maintainer file and let the Maureen or Nash decide if they accept it? |
@kartben thank you for the approval. Can this PR be merged now, or what does "You're not authorized to push to this branch." mean? |
I would encourage a few of you to first step up as Contributors -- this way your reviews will start actually counting (green mark vs. grey mark), and it's one step closer to becoming maintainer(s) Next step would be for WE to identify who would want to be the maintainer (and potentially identify a couple collaborators) and then send a PR updating the MAINTAINERS.yml file indeed. |
Only release managers have the permissions to merge, and pull requests typically need to be in review for a minimum of 2 business days. |
correct board name in documentation