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

Skip to content

cleanup of reporting-related API docstrings - #2241

Merged
rcap107 merged 1 commit into
skrub-data:mainfrom
lisaleemcb:docs_reporting_cleanup
Aug 18, 2026
Merged

cleanup of reporting-related API docstrings#2241
rcap107 merged 1 commit into
skrub-data:mainfrom
lisaleemcb:docs_reporting_cleanup

Conversation

@lisaleemcb

Copy link
Copy Markdown
Contributor

Several small fixes to the reporting API (e.g. the TableReport):

  • fixed two parameter names that were incorrectly specified in the docstring
  • cleaned up some of the wording (in order to improve clarity and reduce redundancy)
  • a few small formatting fixes

@lisaleemcb
lisaleemcb requested a review from emassoulie August 4, 2026 14:21

@jeromedockes jeromedockes left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks great, thanks!

@lisaleemcb
lisaleemcb marked this pull request as ready for review August 11, 2026 12:06

@rcap107 rcap107 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the PR!

@rcap107
rcap107 merged commit 730b535 into skrub-data:main Aug 18, 2026
29 checks passed
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.

3 participants