search

napari / midi-app-controller

Control napari with a MIDI controller.
BSD 3-Clause "New" or "Revised" License
1 stars 3 forks source link

Add more ruff rules #84

Closed lukaszsmolinski closed 6 months ago

lukaszsmolinski commented 6 months ago

It mostly changed the order of imports and too long lines.

Depends on #83 and should be merged after it.

codecov[bot] commented 6 months ago

Codecov Report

Attention: Patch coverage is 81.63265% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 49.14%. Comparing base (68b3eba) to head (a716db7).

Files Patch % Lines
midi_app_controller/gui/binds_editor.py 0.00% 5 Missing :warning:
midi_app_controller/gui/midi_status.py 0.00% 4 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #84 +/- ## ========================================== - Coverage 49.62% 49.14% -0.48% ========================================== Files 15 15 Lines 1066 1056 -10 ========================================== - Hits 529 519 -10 Misses 537 537 ```

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

lukaszsmolinski commented 6 months ago

The other reviewers did not express a willingness to conduct the review.