-
Notifications
You must be signed in to change notification settings - Fork 335
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 process to guide team practice #2174
Conversation
Do we need to change |
@EoinShaughnessy I think we can do that separately - it might take us a while to do the rename and we probably don't want to update the docs until it's done, so I wouldn't want us to block this PR 🙂 |
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.
Thanks @EoinShaughnessy 🎉 I think these changes will really help us to streamline the release process. Noticed a couple of small things that just need tidying up.
@hannalaakso Thanks for the excellent review! I think I've actioned your feedback now. If things look ok, I'll add in the final commits and then get to squashing. |
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.
Thanks for addressing all my comments @EoinShaughnessy, this looks good to me 🙏 I've given it an approval but squashing the commits before it's merged sounds great.
0f5b672
to
7d9a3c4
Compare
Addresses #2175 (Map release process and identify pain points).
We recently had a retro to identify pain points in our release process.
This pull request contains revised steps, based on team feedback, to the doc for our release process. For example, that no major changes should go into a release after an agreed cutoff-point.
Hopefully these revisions will make our future releases run more smoothly.