Skip to content
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

Admin: Ruff formatting #8445

Merged
merged 1 commit into from
Dec 29, 2024
Merged

Conversation

bblommers
Copy link
Collaborator

@bblommers bblommers commented Dec 28, 2024

Ruff complains about the formatting of a type: ignore all of a sudden.

I've tried upgrading ruff, which would improve the formatting - but the new formatting is not compatible with Python 3.8. So we can't do that yet.

@bblommers bblommers added this to the 5.0.26 milestone Dec 28, 2024
@bblommers bblommers force-pushed the admin/ruff-formatting branch 2 times, most recently from e151ffc to c9cc24f Compare December 29, 2024 12:01
@bblommers bblommers force-pushed the admin/ruff-formatting branch from c9cc24f to 661b2b5 Compare December 29, 2024 12:17
Copy link

codecov bot commented Dec 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.62%. Comparing base (5ab97e5) to head (661b2b5).
Report is 10 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8445      +/-   ##
==========================================
- Coverage   94.60%   92.62%   -1.98%     
==========================================
  Files        1168     1225      +57     
  Lines      101707   105633    +3926     
==========================================
+ Hits        96220    97843    +1623     
- Misses       5487     7790    +2303     
Flag Coverage Δ
servertests 27.75% <0.00%> (-1.07%) ⬇️
unittests 92.59% <100.00%> (-1.98%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bblommers bblommers marked this pull request as ready for review December 29, 2024 13:15
@bblommers bblommers merged commit d56c961 into getmoto:master Dec 29, 2024
52 of 53 checks passed
@bblommers bblommers deleted the admin/ruff-formatting branch December 29, 2024 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant