search

django-commons / drf-excel

An XLSX spreadsheet renderer for Django REST Framework.
BSD 3-Clause "New" or "Revised" License
217 stars 40 forks source link

Modernize a bit the codebase with Ruff #101

Closed browniebroke closed 2 weeks ago

browniebroke commented 2 weeks ago

Enable more Ruff rules and fix issues raised

codecov-commenter commented 2 weeks ago

:warning: Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@4a01c06). Learn more about missing BASE report.

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

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #101 +/- ## ======================================= Coverage ? 92.89% ======================================= Files ? 4 Lines ? 352 Branches ? 0 ======================================= Hits ? 327 Misses ? 25 Partials ? 0 ```

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

FlipperPA commented 2 weeks ago

LGTM, whenever you're confident!