Skip to content

DOC Ensures that sklearn.model_selection._split.check_cv passes numpydoc validation#22778

Merged
GaelVaroquaux merged 1 commit intoscikit-learn:mainfrom
chagaz:check_cv_docstring
Mar 12, 2022
Merged

DOC Ensures that sklearn.model_selection._split.check_cv passes numpydoc validation#22778
GaelVaroquaux merged 1 commit intoscikit-learn:mainfrom
chagaz:check_cv_docstring

Conversation

@chagaz
Copy link
Copy Markdown
Contributor

@chagaz chagaz commented Mar 12, 2022 •
edited
Loading

Addresses #21350

  • Added a period at the end of summary
  • Fixed YD01 as described

#pariswimlds

GaelVaroquaux
GaelVaroquaux approved these changes Mar 12, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@GaelVaroquaux GaelVaroquaux left a comment

Choose a reason for hiding this comment

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

This Looks Good To Me

@GaelVaroquaux
Copy link
Copy Markdown
Member

GaelVaroquaux commented Mar 12, 2022

This is a doc-only change. Merging with one review.

@GaelVaroquaux GaelVaroquaux merged commit babc517 into scikit-learn:main Mar 12, 2022
@thomasjpfan thomasjpfan mentioned this pull request Mar 12, 2022
Closed
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Apr 6, 2022
@chagaz @glemaitre
Fixed sklearn.model_selection._split.check_cv docstring in scikit-lea…
4c01ad9 
…rn#21350 (scikit-learn#22778)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@GaelVaroquaux GaelVaroquaux GaelVaroquaux approved these changes

Assignees

No one assigned

Labels

Documentation module:model_selection No Changelog Needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@chagaz @GaelVaroquaux