Skip to content

DOC Fix parametrize_with_checks link#17263

Merged
rth merged 1 commit intomasterfrom
fix-parametrize-with-checks-link
May 18, 2020
Merged

DOC Fix parametrize_with_checks link#17263
rth merged 1 commit intomasterfrom
fix-parametrize-with-checks-link

Conversation

@rth
Copy link
Copy Markdown
Member

@rth rth commented May 18, 2020

The link to parametrize_with_checks under "Rolling your own estimator" is currently not working
https://scikit-learn.org/dev/developers/develop.html#rolling-your-own-estimator

@thomasjpfan thomasjpfan modified the milestone: 0.23.1 May 18, 2020
@thomasjpfan
Copy link
Copy Markdown
Member

I have always been unsure if we should include these types of fixes into the bugfix release, so the website gets updated.

What do you think?

@NicolasHug NicolasHug added this to the 0.23.1 milestone May 18, 2020
Copy link
Copy Markdown
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

Thanks @rth

I have always been unsure if we should include these types of fixes into the bugfix release, so the website gets updated.

yeah I think we should

@rth
Copy link
Copy Markdown
Member Author

rth commented May 18, 2020

I also think that we should, even independently of minor bugfix releases for the documentation.

@rth rth merged commit 29f83f8 into master May 18, 2020
@rth rth deleted the fix-parametrize-with-checks-link branch May 18, 2020 15:39
adrinjalali pushed a commit to adrinjalali/scikit-learn that referenced this pull request May 18, 2020
viclafargue pushed a commit to viclafargue/scikit-learn that referenced this pull request Jun 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants