refactor(ci): bump ruff target version#8048
refactor(ci): bump ruff target version#8048maxrabin wants to merge 3 commits intoaws-powertools:developfrom
Conversation
This aligns with the Poetry defined minimum version. Marking this with ! for Breaking Change anyways as this will break the code on python versions 3.8 and 3.9 as Ruff will update all syntax and libraries to 3.10+-only compatibilty
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #8048 +/- ##
===========================================
- Coverage 96.65% 96.61% -0.05%
===========================================
Files 282 282
Lines 13775 13523 -252
Branches 1096 1096
===========================================
- Hits 13314 13065 -249
+ Misses 339 336 -3
Partials 122 122 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Hey @maxrabin, thanks for putting this together - I like the direction of modernizing the typing syntax ( However, I have a few concerns:
I'm going to close this PR for now, but the issue itself makes total sense I'll keep the issue open and work on splitting this into smaller, reviewable PRs that we can merge incrementally without breaking anything. If you'd like to be part of this effort, you're more than welcome! Once I split the issue into smaller scoped tasks, feel free to pick any of them up. Thanks a lot for another amazing idea and contribution. |
|
Totally appreciate your concerns, this is obviously something that has to be done carefully. I'd be very happy to be part of the effort, keep me posted. I'm a huge fan of this project and I make extensive use of it in every single lambda function i write! |
Awesome, glad to hear that! It makes supper happy to know that I'm helping customers in their workoads. I'll split the issue into smaller tasks in the next few days and ping you here so you can pick up whatever interests you. Thanks again! |



Issue number: closes #8047
Summary
Update Ruff target Python version from 3.8 to 3.10 to align with version targets defined in Poetry.
Then fix/implement all Ruff lint issues.
Changes
Mostly changes around typing (List -> list, Optional[str] -> str | None)
User experience
Code is more modern and standardized
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.