Skip to content

MNT Fix doctest dict value #31340

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 12, 2025
Merged

Conversation

lesteve
Copy link
Member

@lesteve lesteve commented May 9, 2025

Currently scipy-doctest has a bug in dict comparison reported in scipy/scipy_doctest#195 and will be fixed in scipy/scipy_doctest#196.

We may as well fix this doctest now.

Copy link

github-actions bot commented May 9, 2025

✔️ Linting Passed

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

Generated for commit: 5be9e0b. Link to the linter CI: here

Copy link
Member

@lucyleeow lucyleeow left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@lucyleeow lucyleeow merged commit acb3833 into scikit-learn:main May 12, 2025
39 checks passed
@lesteve lesteve deleted the dict-doctest branch May 12, 2025 06:50
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.

3 participants