search

boto / s3transfer

Amazon S3 Transfer Manager for Python
Apache License 2.0
209 stars 133 forks source link

Ruff update #308

Closed nateprewitt closed 2 months ago

nateprewitt commented 3 months ago

This is a follow up to boto/boto3#4161 to move our repos to use ruff in favor of our existing pre-commit cocktail.

This should be largely inline with our existing configuration with some minor changes to fix f-string upgrades missed with our previous setup and some newline fixes. This PR keeps our existing quote preservation behavior until we get all projects moved to ruff. At that point, we can look at turning it to "double".

codecov-commenter commented 3 months ago

Codecov Report

Attention: Patch coverage is 90.90909% with 1 line in your changes missing coverage. Please review.

Project coverage is 82.44%. Comparing base (123ff16) to head (a414253). Report is 4 commits behind head on develop.

Files Patch % Lines
s3transfer/futures.py 50.00% 1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #308 +/- ## =========================================== - Coverage 83.08% 82.44% -0.65% =========================================== Files 16 16 Lines 2814 2836 +22 =========================================== Hits 2338 2338 - Misses 476 498 +22 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.