Skip to content

Conversation

@rouk1
Copy link
Contributor

@rouk1 rouk1 commented Jun 2, 2025

During my journey through this MOOC I took a few note. The easy fix I can contribute are in this PR.

Copy link
Collaborator

@ogrisel ogrisel left a comment

Choose a reason for hiding this comment

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

Thanks for the fixes, LGTM. Let's see if the CI is green and merge.

@rouk1
Copy link
Contributor Author

rouk1 commented Jun 2, 2025

Hopefully it will now that I ran pre-commit 😅

@ogrisel
Copy link
Collaborator

ogrisel commented Jun 2, 2025

Thanks for the quick follow-up, I was about to fix it myself. Re-running the CI.

@ogrisel ogrisel merged commit 1c40b1c into INRIA:main Jun 2, 2025
3 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 2, 2025
)

* minor improvment in wording and import statement order

* use pre-commit hooks 1c40b1c
@rouk1 rouk1 deleted the minor-improvments branch June 2, 2025 17:04
ArturoAmorQ pushed a commit to ArturoAmorQ/scikit-learn-mooc that referenced this pull request Oct 23, 2025
* minor improvment in wording and import statement order

* use pre-commit hooks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants