-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
DOC Update maintainers' guide to use new wheel builder system #18878
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
DOC Update maintainers' guide to use new wheel builder system #18878
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.
LGTM, just a few suggestions.
It's nice to see a PR that (legitimately) removes lines in the maintainers' documentation ;)
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.
The github checklist regarding
update dependencies and release tag at https://github.com/MacPython/scikit-learn-wheels
should be removed?
Since we are not longer using the |
Yes. |
I have applied the suggested changes 😉. |
…-learn into update_mainteiners_guide
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.
LGTM
Reference Issues/PRs
Closes #18776.
What does this implement/fix? Explain your changes.
This PR updates the maintainers' guide to use the
Wheel builder
workflow.Any other comments?
CC @ogrisel.