-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Update release managers group in documentation #4555
Conversation
The existing group was not valid. This may have been updated at some point. This ensures the documentation is pointed to the correct group.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
@jtslear can you run a 'make generate' |
All of the go is rubbing off on me! I dropped the Thanks for updating the reference! I'll approve this once the yaml is regenerated, as @markyjackson-taulia mentioned. |
@jtslear -- Can you check in on this? The PR doesn't have any content right now. /hold |
@justaugustus it would appear that the |
Hehe, no worries, John. Thanks again for the fix! |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cblecker, jtslear, justaugustus, markyjackson-taulia The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
The existing group was not valid. This may have been updated at some point. This ensures the documentation is pointed to the correct group.