-
Notifications
You must be signed in to change notification settings - Fork 392
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
Move Major Themes responsibility from Release Notes to Comms #2492
Conversation
/hold |
/assign @a-mccarthy Since you are the incoming Comms lead, this will affect you. Thoughts? |
Sounds good to me. I think it makes sense to move over the responsibility to comms since we are the primary folks using this content for our release deliverables. 👍 |
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.
I agree with this proposal.
Thanks, Kat for organizing this.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: katcosgrove, ramrodo 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 |
/unhold Thanks y'all! |
Late to the party, but completely in agreement with the proposal, this was one of the suggestions I had for the 1.29 retro since the current situation is an artefact of how things were done in the beginning (when Release Notes were the single source of updates for a release, before there was a release blog), this suggestions greatly clarifies things and puts them in line with current practices. /lgtm |
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.
@
What type of PR is this:
/kind cleanup
/kind documentation
What this PR does / why we need it:
The collection of Major Themes is currently handled by Release Notes, when it is more in line with the duties of the Comms team and most prominently consumed by the Comms team. To better reflect the actual behavior of the teams, I have moved the primary responsibility for collection of Major Themes to Comms, and put Release Notes in a supporting role for this duty.
Which issue(s) this PR fixes:
None
Special notes for your reviewer: