Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@glemaitre
Copy link
Member

This is a follow-up of #29877 that is adding a test and an entry in the changelog to acknowledge a potential bug.

I raised the point here: #29877 (comment)

@glemaitre
Copy link
Member Author

ping @adrinjalali @OmarManzoor I think that this is an easy follow-up.

@github-actions
Copy link

github-actions bot commented Sep 18, 2024

✔️ Linting Passed

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

Generated for commit: 87c8f45. 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

@OmarManzoor OmarManzoor added the Quick Review For PRs that are quick to review label Sep 19, 2024
@adrinjalali adrinjalali enabled auto-merge (squash) September 19, 2024 07:53
@adrinjalali adrinjalali merged commit cfa5270 into scikit-learn:main Sep 19, 2024
kbharat1210 pushed a commit to kbharat1210/scikit-learn that referenced this pull request Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:linear_model Quick Review For PRs that are quick to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants