-
Notifications
You must be signed in to change notification settings - Fork 62
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
docs: add release cadence docs, and tasks for notifying after release #823
Conversation
Signed-off-by: Xander Grzywinski <[email protected]>
f1e8b8c
to
27cf870
Compare
docs/docs/release-management.md
Outdated
|
||
**Minor Releases** | ||
|
||
- X.Y.0-alpha.W, W >= 0 (Branch : master) |
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.
- X.Y.0-alpha.W, W >= 0 (Branch : master) | |
- X.Y.0-alpha.W, W >= 0 (Branch: main) |
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.
applies below too
docs/docs/release-management.md
Outdated
|
||
**Minor Releases** | ||
|
||
- X.Y.0-alpha.W, W >= 0 (Branch : master) |
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 think we are okay to skip alpha. Since we said "released as needed", we can keep in the doc.
## Notifying | ||
|
||
1. Send an email to the [Eraser mailing list](https://groups.google.com/g/eraser-dev) announcing the release, with links to GitHub. |
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.
this is a good idea, it would be cool to automate these. obviously, out of scope for 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.
it can be a reusable generic automation
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.
minor nit, otherwise LGTM
Signed-off-by: Xander Grzywinski <[email protected]>
What this PR does / why we need it:
Add release cadence docs. Also add a couple items to the release process to include publicizing the release.
Which issue(s) this PR fixes (optional, using
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when the PR gets merged):Fixes #822
Special notes for your reviewer: