-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Added versioning documentation #5117
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
Conversation
RELEASING.md
Outdated
| security, installation or critical bug fixes. These are less common as it is preferred | ||
| to stick to quarterly releases. | ||
|
|
||
| Between quarterly releases, ".dev0" is appended to the version, indicating that this is |
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.
| Between quarterly releases, ".dev0" is appended to the version, indicating that this is | |
| Between quarterly releases, ".dev0" is appended to the `master` version, indicating that this is |
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.
Let me know if you disagree, but I've changed it to 'appended to the master branch'
Co-authored-by: Hugo van Kemenade <[email protected]>
Co-authored-by: Hugo van Kemenade <[email protected]>
|
Perhaps this should be its own page/section somewhere at https://pillow.readthedocs.io? It'd be nice to keep RELEASING.md a bit cleaner for the release manager. We can link from RELEASING.md and https://pillow.readthedocs.io/en/stable/releasenotes/index.html to the new place. |
ad26366 to
54a7de1
Compare
|
Ok, there's a link at https://pillow--5117.org.readthedocs.build/en/5117/releasenotes/index.html to the page. |
54a7de1 to
d13a3ce
Compare
|
LGTM, thanks all! |
Co-authored-by: Hugo van Kemenade <[email protected]>
Resolves #5003, by essentially adding a restructured version of #5003 (comment) to RELEASING.md.