Skip to content

Ruff linter: enable rule UP031 #13049

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Ruff linter: enable rule UP031 #13049

wants to merge 3 commits into from

Conversation

giograno
Copy link
Member

@giograno giograno commented Aug 22, 2025

Motivation

#12953 introduced the pyupgrade rules to our ruff linter.
However, a few rules have been ignored. This PR enabled UP031 and fixes the code accordingly (the latter had no violations).

@giograno giograno added this to the Playground milestone Aug 22, 2025
@giograno giograno self-assigned this Aug 22, 2025
@giograno giograno added semver: patch Non-breaking changes which can be included in patch releases skip-docs Pull request does not require documentation changes labels Aug 22, 2025
Copy link

S3 Image Test Results (AMD64 / ARM64)

    2 files    2 suites   8m 10s ⏱️
  517 tests 467 ✅  50 💤 0 ❌
1 034 runs  934 ✅ 100 💤 0 ❌

Results for commit 83574a6.

Copy link

Test Results - Preflight, Unit

22 144 tests  ±0   20 407 ✅ ±0   6m 15s ⏱️ -11s
     1 suites ±0    1 737 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 83574a6. ± Comparison against base commit 047b419.

Copy link

Test Results (amd64) - Acceptance

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

Results for commit 83574a6. ± Comparison against base commit 047b419.

Copy link

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 20m 50s ⏱️
5 003 tests 4 403 ✅ 600 💤 0 ❌
5 009 runs  4 403 ✅ 606 💤 0 ❌

Results for commit 83574a6.

Copy link

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 42m 21s ⏱️ - 2m 37s
4 632 tests ±0  4 190 ✅ ±0  442 💤 ±0  0 ❌ ±0 
4 634 runs  ±0  4 190 ✅ ±0  444 💤 ±0  0 ❌ ±0 

Results for commit 83574a6. ± Comparison against base commit 047b419.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: patch Non-breaking changes which can be included in patch releases skip-docs Pull request does not require documentation changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant