DOC simplify ToFloat import example - #2212
Conversation
|
Hi, thank you for reviewing. I checked the failing CI jobs. Both The error mentions that Since this PR only updates one docstring import example from a private module to the public API, I think the CI failure may be unrelated to this change. Please let me know if there is anything I should update from my side. |
|
Thank you for this contribution @Khadija-de ! indeed there seems to be a temporary failure with the package index from which the CI installs some packages for some of the jobs, it is unrelated to your change and affects other Pull Requests as well. We will re-run the CI job once it is resolved |
|
All good! The CI is running. |
rcap107
left a comment
There was a problem hiding this comment.
Thanks a lot for this PR @Khadija-de
Fixes #2130.
This updates the ToFloat docstring example to import ToFloat from the public skrub API instead of the private skrub._to_float module.