Repository navigation
Bump urllib3, pyjwt, cryptography and aiohttp; ruff cleanup - #91
Merged
Merged
Conversation
Bumps [aiohttp](https://github.com/aio-libs/aiohttp) from 3.14.1 to 3.14.3. - [Changelog](https://github.com/aio-libs/aiohttp/blob/master/CHANGES.rst) - [Commits](aio-libs/aiohttp@v3.14.1...v3.14.3) --- updated-dependencies: - dependency-name: aiohttp dependency-version: 3.14.3 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [cryptography](https://github.com/pyca/cryptography) from 49.0.0 to 50.0.0. - [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst) - [Commits](pyca/cryptography@49.0.0...50.0.0) --- updated-dependencies: - dependency-name: cryptography dependency-version: 50.0.0 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [pyjwt](https://github.com/jpadilla/pyjwt) from 2.13.0 to 2.15.0. - [Release notes](https://github.com/jpadilla/pyjwt/releases) - [Changelog](https://github.com/jpadilla/pyjwt/blob/master/CHANGELOG.rst) - [Commits](jpadilla/pyjwt@2.13.0...2.15.0) --- updated-dependencies: - dependency-name: pyjwt dependency-version: 2.15.0 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [urllib3](https://github.com/urllib3/urllib3) from 2.7.0 to 2.8.0. - [Release notes](https://github.com/urllib3/urllib3/releases) - [Changelog](https://github.com/urllib3/urllib3/blob/main/CHANGES.rst) - [Commits](urllib3/urllib3@2.7.0...2.8.0) --- updated-dependencies: - dependency-name: urllib3 dependency-version: 2.8.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
… test/dependabot-all
…0' into test/dependabot-all
…to test/dependabot-all
Missing newline at end of file, unused imports, import order and redundant open() modes (ruff check --fix).
Use identity checks (is True / is False) instead of == comparisons, keeping the assertions strict, and rename the ambiguous variable l.
@classproperty passes the class, so check its methods for cls like @classmethod (N805), and allow its lowercase decorator-style name (N801).
byjg
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Combines the open Dependabot PRs into a single merge:
requests,docker,kubernetes(runtime)pyOpenSSLincertbot.py(runtime), E2E testskubernetesclient (runtime)Only
uv.lockchanges. The branches merged without conflicts anduv lock --checkpasses.cryptography 50 is a major bump. pyOpenSSL 26.4.0 accepts
cryptography>=49,<51, and the certbot tests that create real certificates and load them throughcrypto.load_certificatepass.Tested locally
The failing check on #82 was
Tests-E2E-Swarmtiming out while pullinghaproxy:3.3-alpinefrom Docker Hub, not the aiohttp bump.Merging with a merge commit (not squash) keeps the Dependabot commits in
master, so GitHub marks #82, #83, #88 and #89 as merged.Ruff cleanup
Also fixes the existing ruff errors (63 → 5). These are code-style changes only; unit tests still pass (183).
a3c40edAuto-fixes (ruff check --fix): missing newline at end of file, unused imports, import order, redundantopen()modes.302d6f9E712/E741 in tests:is True/is Falseinstead of==, which keeps the assertions strict, andl→line.fcb5da7pyproject.toml:@classpropertypasses the class, soclassmethod-decoratorsmakes ruff expectclslike@classmethod(N805, and it still flagsself), andextend-ignore-namesallows the lowercase decorator name (N801) while keeping the default ignore list.The 5 remaining F841 (unused variables in tests) are left for a follow-up.