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
Chore: update release template #1350
Conversation
dfde12c
to
b4cb8e0
Compare
@thekaveman @machikoyasuda Can you read over this and tell me if this list makes sense to you? |
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.
You're right, this is harder than I expected!
I think the "staging target" thing is needlessly confusing and we can assume by the time we're releasing, dev
has already been squared away / secrets configured / QA'd etc.
Also, I think in #1284 we decided that the steps for hotfix are different enough that they warrant docs of their own; so...
Suggestions:
- Reformat into 2 lists: one for
regular
, one forhotfix
- Make the first step in
regular
something like "Ensure thedev
branch and secrets are up to date", then bump the version etc. (i.e. we don't need individual steps for updating/refreshing dev secrets) - Remove all the "if applicable" language
This feels overly complicated because we're trying to represent 2 flows in 1 list.
also consolidate steps related to ensuring secrets are up to date
Ready for re-review, whenever convenient |
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.
Looks good. I don't think we need this extra step/check-box but not a deal breaker.
remove item that was meant to explain motivation of subsequent steps Co-authored-by: Kegan Maher <kegan@compiler.la>
@thekaveman I settled on removing that extra step 😄 thanks for all your feedback on simplifying this template! |
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.
🚀
Closes #1333
Closes #1284
This PR updates the release issue template to include steps for ensuring secrets are up to date when releasing / deploying the Benefits app.
To simplify the flow of steps, we split the checklist out into two checklists, one for each type of release (Regular and Hotfix).
This PR also updates documentation on "Making a release" to reflect our hotfix checklist.