X Tutup
Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

Update ruff target-version to 3.13#13688

Merged
giograno merged 7 commits intomainfrom
ruff-to-313
Feb 4, 2026
Merged

Update ruff target-version to 3.13#13688
giograno merged 7 commits intomainfrom
ruff-to-313

Conversation

@giograno
Copy link
Member

@giograno giograno commented Feb 3, 2026

Motivation

While making InterruptiblePriorityQueue generic (see #13684), I was using the newer Python 3.12 syntax, and I was getting linting errors. Therefore, I noticed that ruff points to 311 as the target version, which does not match .python-version.

Changes

  • Move ruff target version to 313.
  • Fix a few linting issues, most notably around the new generic syntax.

@giograno giograno added semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Feb 3, 2026
@github-actions
Copy link

github-actions bot commented Feb 3, 2026

S3 Image Test Results (AMD64 / ARM64)

    2 files      2 suites   8m 7s ⏱️
  559 tests   507 ✅  52 💤 0 ❌
1 118 runs  1 014 ✅ 104 💤 0 ❌

Results for commit 5e0ea6f.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Test Results - Preflight, Unit

23 114 tests  ±0   21 255 ✅ ±0   6m 47s ⏱️ +30s
     1 suites ±0    1 859 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 5e0ea6f. ± Comparison against base commit 2a499fd.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 3s ⏱️ ±0s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 5e0ea6f. ± Comparison against base commit 2a499fd.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Test Results - Alternative Providers

992 tests   596 ✅  21m 39s ⏱️
  2 suites  396 💤
  2 files      0 ❌

Results for commit 5e0ea6f.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 38m 32s ⏱️
5 610 tests 5 049 ✅ 561 💤 0 ❌
5 616 runs  5 049 ✅ 567 💤 0 ❌

Results for commit 5e0ea6f.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 59m 0s ⏱️ +17s
5 189 tests ±0  4 798 ✅ ±0  391 💤 ±0  0 ❌ ±0 
5 191 runs  ±0  4 798 ✅ ±0  393 💤 ±0  0 ❌ ±0 

Results for commit 5e0ea6f. ± Comparison against base commit 2a499fd.

♻️ This comment has been updated with latest results.

@giograno giograno added this to the 4.14 milestone Feb 4, 2026
@giograno giograno marked this pull request as ready for review February 4, 2026 06:57
Copy link
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

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

Good catch and nice cleanup! 🤩

@giograno giograno merged commit 26619a9 into main Feb 4, 2026
52 checks passed
@giograno giograno deleted the ruff-to-313 branch February 4, 2026 09:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

X Tutup