Improved the documentation of Cleaner#1811
Conversation
|
Hi @Faith-Nchifor, thank for opening the PR. At the moment, the CI is failing because the formatting is incorrect. In order to fix that problem, you will have to run pre-commit checks in the environment that you are using to develop skrub. pre-commit run --all-filesand then commit the modified files. |
|
I think it would be useful to add the |
|
Alright. |
|
Hey @rcap107 |
|
Hi @Faith-Nchifor, I made a small additional change to the PR adding a cross-reference in the list of steps done by the cleaner. I'll merge the PR once the tests pass. Thanks a lot! |
Co-authored-by: Riccardo Cappuzzo <riccardo.cappuzzo@gmail.com>
Co-authored-by: Riccardo Cappuzzo <riccardo.cappuzzo@gmail.com>
As requested in issue #1785
I included the
DropUninformativein thesee alsosection of the Cleaner documentation.I also built the documentation, and there were no errors.