Skip to content

Conversation

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.8.6 → v0.11.4](astral-sh/ruff-pre-commit@v0.8.6...v0.11.4)
- [github.com/psf/black-pre-commit-mirror: 24.10.0 → 25.1.0](psf/black-pre-commit-mirror@24.10.0...25.1.0)
- [github.com/PyCQA/flake8: 7.1.1 → 7.2.0](PyCQA/flake8@7.1.1...7.2.0)
- [github.com/PyCQA/pylint: v3.3.3 → v3.3.6](pylint-dev/pylint@v3.3.3...v3.3.6)
- [github.com/pre-commit/mirrors-mypy: v1.14.1 → v1.15.0](pre-commit/mirrors-mypy@v1.14.1...v1.15.0)
- [github.com/pycqa/isort: 5.13.2 → 6.0.1](PyCQA/isort@5.13.2...6.0.1)
@github-actions github-actions bot added ⚙️ dependencies affected functionality: dependencies 🛠 code-quality change type: code-quality labels Apr 7, 2025
@Bibo-Joshi
Copy link
Member

ruff suggests using typing.Self in a lot of places, but that was introduced only in py3.11 while we're supporting 3.9+. I've left a comment astral-sh/ruff#9761 (comment). Let's give them a bit to reply :)

@Bibo-Joshi
Copy link
Member

astral-sh/ruff#9761 has been closed by astral-sh/ruff#17611 which should hopefully be included in the next release. I'm subscribing to those for now.

let's see if this works, it did not work for me locally
@Bibo-Joshi
Copy link
Member

I can't get the new ruff stetting to work. I've openend the follow-up issue astral-sh/ruff#17821

@Bibo-Joshi
Copy link
Member

astral-sh/ruff#17821 already got closed 🤯 I've verified that the fix works on our code base. Waiting for the next release again :D

@Bibo-Joshi Bibo-Joshi merged commit c34e19e into master May 12, 2025
19 of 25 checks passed
@Bibo-Joshi Bibo-Joshi deleted the pre-commit-ci-update-config branch May 12, 2025 19:23
@github-actions github-actions bot locked and limited conversation to collaborators May 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

🛠 code-quality change type: code-quality ⚙️ dependencies affected functionality: dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants