8000 Bump versions of pre-commit hooks by illia-v · Pull Request #3716 · urllib3/urllib3 · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@illia-v
Copy link
Member
@illia-v illia-v commented Nov 19, 2025

https://github.com/urllib3/urllib3/actions/runs/19516465216/job/55869409115?pr=3715

There is a warning in the lint output that the isort step may stop working soon:

Warning: repo [https://github.com/PyCQA/isort](https://github.com/PyCQA/isort%60) uses deprecated stage names (commit, merge-commit, push) which will be removed in a future version. Hint: often pre-commit autoupdate --repo [https://github.com/PyCQA/isort](https://github.com/PyCQA/isort%60) will fix this. if it does not -- consider reporting an issue to that repo.

Additionally, my editor started reformatting some files with a newer version of black and isort causing noisy git diffs.

So I'm upgrading all pre-commit hooks related to Python here and applying suggested fixes.

warning[dependabot-cooldown]: insufficient cooldown in Dependabot updates
 --> ./.github/dependabot.yml:3:5
  |
3 |   - package-ecosystem: "github-actions"
  |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ missing cooldown configuration
  |
  = note: audit confidence → High
  = note: this finding has an auto-fix
src/urllib3/util/request.py:231:13: F824 `nonlocal body` is unused: name is never assigned in scope
src/urllib3/util/request.py:231:13: F824 `nonlocal blocksize` is unused: name is never assigned in scope
@illia-v illia-v added the Skip Changelog Pull requests that don't require a changelog entry label Nov 19, 2025
@illia-v illia-v mentioned this pull request Nov 20, 2025
illia-v and others added 2 commits November 21, 2025 14:54
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@illia-v illia-v merged commit 1f6abac into urllib3:main Nov 28, 2025
42 checks passed
@illia-v illia-v deleted the pre-commit-2025-11 branch November 28, 2025 11:43
@hugovk
Copy link
Contributor
hugovk commented Nov 28, 2025

Tip: you may want to enable https://pre-commit.ci/ for this repo to have automated updates of the hooks. It can also update PRs when there are fixes, which can save a bit of extra hassle.

I also stick this at the bottom of the config so you only get autoupdates quarterly instead of the default weekly:

ci:
  autoupdate_schedule: quarterly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog Pull requests that don't require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0