-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
DOC minor updates to release process #21172
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
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.
Otherwise LGTM
@@ -211,6 +211,12 @@ Making a release | |||
|
|||
https://github.com/scikit-learn/scikit-learn/actions?query=workflow%3A%22Publish+to+Pypi%22 | |||
|
|||
4.1 You can test the conda-forge builds by submitting a PR to the feedstock | |||
repo: https://github.com/conda-forge/scikit-learn-feedstock. If you want to | |||
publish an RC release on conda-forge, the PR should target the `rc` branch |
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.
Could we make it in bold regarding the rc
branch. I think this is something that could be easily overlooked.
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 personally don't check a rendered version of this doc, so I'm not sure making it bold would help.
Let's merge as-is. It anyway a great improvement. |
Co-authored-by: Guillaume Lemaitre <g.lemaitre58@gmail.com>
Co-authored-by: Guillaume Lemaitre <g.lemaitre58@gmail.com>
Co-authored-by: Guillaume Lemaitre <g.lemaitre58@gmail.com>
updates the release process, a few minor changes according to my latest experience.