Skip to content

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Oct 7, 2024

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.5.0 → v0.6.9](astral-sh/ruff-pre-commit@v0.5.0...v0.6.9)
- [github.com/jendrikseipp/vulture: v2.11 → v2.13](jendrikseipp/vulture@v2.11...v2.13)
- [github.com/pre-commit/pre-commit-hooks: v4.6.0 → v5.0.0](pre-commit/pre-commit-hooks@v4.6.0...v5.0.0)
- [github.com/asottile/pyupgrade: v3.16.0 → v3.17.0](asottile/pyupgrade@v3.16.0...v3.17.0)
- [github.com/sphinx-contrib/sphinx-lint: v0.9.1 → v1.0.0](sphinx-contrib/sphinx-lint@v0.9.1...v1.0.0)
- [github.com/pre-commit/mirrors-clang-format: v18.1.8 → v19.1.1](pre-commit/mirrors-clang-format@v18.1.8...v19.1.1)
@mroeschke mroeschke requested a review from rhshadrach as a code owner October 7, 2024 18:42
@mroeschke mroeschke added the Code Style Code style, linting, code_checks label Oct 7, 2024
@rhshadrach
Copy link
Member

/preview

@mroeschke mroeschke added this to the 3.0 milestone Oct 7, 2024
@mroeschke
Copy link
Member

/preview

Copy link
Contributor

github-actions bot commented Oct 7, 2024

Website preview of this PR available at: https://pandas.pydata.org/preview/pandas-dev/pandas/59998/

Copy link
Member

@rhshadrach rhshadrach left a comment

Choose a reason for hiding this comment

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

Looks good - on the Styler page a few of the code boxes need side scrolling on my browser. This is also true, and to a more severe extent, on main.

},
"outputs": [],
"source": [
"import matplotlib.pyplot\n",
Copy link
Member

@rhshadrach rhshadrach Oct 7, 2024

Choose a reason for hiding this comment

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

This cell can now be removed, if it is indeed unnecessary. I didn't notice anything off with this change.

Copy link
Member

@rhshadrach rhshadrach left a comment

Choose a reason for hiding this comment

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

lgtm

@mroeschke mroeschke merged commit a0f9140 into main Oct 8, 2024
53 checks passed
@mroeschke mroeschke deleted the pre-commit-ci-update-config branch October 8, 2024 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Code Style Code style, linting, code_checks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants