Skip to content

Conversation

@pre-commit-ci
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Jul 7, 2025

pre-commit-ci bot added 2 commits July 7, 2025 18:14
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.12.1 → v0.12.2](astral-sh/ruff-pre-commit@v0.12.1...v0.12.2)
- [github.com/pycqa/flake8: 7.2.0 → 7.3.0](PyCQA/flake8@7.2.0...7.3.0)
@FBruzzesi FBruzzesi merged commit 8afbeec into main Jul 8, 2025
31 of 32 checks passed
@FBruzzesi FBruzzesi deleted the pre-commit-ci-update-config branch July 8, 2025 21:04
dangotbanned added a commit that referenced this pull request Jul 9, 2025
Following (#2803), two (`PD002`) ignores are no longer needed
- https://github.com/narwhals-dev/narwhals/blob/54bf3d3ec77092d4babc970f245efc2185274e96/narwhals/_pandas_like/group_by.py#L217
- https://github.com/narwhals-dev/narwhals/blob/54bf3d3ec77092d4babc970f245efc2185274e96/narwhals/_pandas_like/group_by.py#L264

`pre-commit` autoupdate enforces the latest version of `ruff` in ci, but we don't have anything syncing this with `pyproject.toml`.
A solution for now is just to leave it unpinned, so at least it'll install the latest when updating a local `venv`
@dangotbanned dangotbanned mentioned this pull request Jul 9, 2025
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