Skip to content

DOC Remove unneeded html_show_search_summary since True is the default #29139

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 1 commit into from
May 31, 2024

Conversation

lesteve
Copy link
Member

@lesteve lesteve commented May 30, 2024

html_show_search_summary is True by default (source).

Also this removes the confusing comment above that does not match the code ...

Copy link

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 8648cbb. Link to the linter CI: here

@lesteve lesteve added the Quick Review For PRs that are quick to review label May 30, 2024
@lesteve
Copy link
Member Author

lesteve commented May 30, 2024

@glemaitre glemaitre merged commit 00601e3 into scikit-learn:main May 31, 2024
34 checks passed
jeremiedbb pushed a commit to jeremiedbb/scikit-learn that referenced this pull request Jul 2, 2024
@jeremiedbb jeremiedbb mentioned this pull request Jul 2, 2024
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Quick Review For PRs that are quick to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants