Skip to content

Conversation

@adrinjalali
Copy link
Member

Fixing this:

image

@adrinjalali
Copy link
Member Author

This PRs result:

image

@adrinjalali adrinjalali added the Quick Review For PRs that are quick to review label Dec 9, 2022
@betatim betatim added the Waiting for Second Reviewer First reviewer is done, need a second one! label Dec 9, 2022
@betatim
Copy link
Member

betatim commented Dec 9, 2022

(not strictly looking for a 2nd reviewer, but we need someone who can press merge)

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

Can we backport this onto the 1.2.X branch so it is also fixed on the stable website?

@thomasjpfan thomasjpfan enabled auto-merge (squash) December 9, 2022 15:43
@adrinjalali adrinjalali added this to the 1.2.1 milestone Dec 9, 2022
@thomasjpfan thomasjpfan merged commit 86fbdb1 into scikit-learn:main Dec 9, 2022
@adrinjalali adrinjalali deleted the nightly branch December 9, 2022 16:01
Vincent-Maladiere pushed a commit to Vincent-Maladiere/scikit-learn that referenced this pull request Dec 14, 2022
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Dec 21, 2022
jjerphan pushed a commit to jjerphan/scikit-learn that referenced this pull request Jan 3, 2023
jjerphan pushed a commit to jjerphan/scikit-learn that referenced this pull request Jan 20, 2023
jjerphan pushed a commit to jjerphan/scikit-learn that referenced this pull request Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation No Changelog Needed Quick Review For PRs that are quick to review Waiting for Second Reviewer First reviewer is done, need a second one!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants