Skip to content

MNT Fix typos in pairwise tests #31651

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
Jun 25, 2025

Conversation

lucyleeow
Copy link
Member

@lucyleeow lucyleeow commented Jun 25, 2025

Reference Issues/PRs

Remove unused test parametrization and unnecessary comment, left from #29822 (sorry!)

What does this implement/fix? Explain your changes.

Any other comments?

@lucyleeow lucyleeow added Array API Quick Review For PRs that are quick to review labels Jun 25, 2025
@lucyleeow
Copy link
Member Author

maybe @ogrisel ?

Copy link

✔️ Linting Passed

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

Generated for commit: 6e8ea12. Link to the linter CI: here

Copy link
Contributor

@OmarManzoor OmarManzoor 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

@OmarManzoor OmarManzoor merged commit 9028b51 into scikit-learn:main Jun 25, 2025
49 of 52 checks passed
@lucyleeow lucyleeow deleted the clean_pairwise branch June 25, 2025 08:32
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.

2 participants