Skip to content

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

Merged
merged 7 commits into from
Nov 26, 2020

Conversation

alfaro96
Copy link
Member

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.

@alfaro96
Copy link
Member Author

Two side notes: We have to update this information when #18800 and #18801 get merged, which should simplify steps 3, 4 and 5.

@alfaro96 alfaro96 changed the title MNT Update maintainers' guide to use new wheel builder system DOC Update maintainers' guide to use new wheel builder system Nov 20, 2020
@alfaro96 alfaro96 added this to the 0.24 milestone Nov 23, 2020
Copy link
Member

@ogrisel ogrisel left a 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 ;)

Copy link
Member

@thomasjpfan thomasjpfan left a 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?

@alfaro96
Copy link
Member Author

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 MacPython/scikit-learn-wheels repository, I think that it should be removed.

@ogrisel
Copy link
Member

ogrisel commented Nov 26, 2020

Since we are not longer using the MacPython/scikit-learn-wheels repository, I think that it should be removed.

Yes.

@alfaro96
Copy link
Member Author

I have applied the suggested changes 😉.

Copy link
Member

@thomasjpfan thomasjpfan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thomasjpfan thomasjpfan merged commit eaa45c8 into scikit-learn:master Nov 26, 2020
@alfaro96 alfaro96 deleted the update_mainteiners_guide branch November 27, 2020 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Updade the maintainers' guide to use the new Wheel Builder github actions tool
3 participants