mantidproject / vesuvio / 15359233589

50%

Build Type

push

github

Committed by web-flow

Commit Message

Run ruff and set it up on pre-commit (#195)

* Remove folder specific to IDE configuration

Aparently this folder is used by the JetBrains IDE for configuration so
it's something not related to the project itself.

* Set up pre-commit to use ruff

* Fix F401

* Fix F811 by remove re-definitions

One was a genuine mistake were two different unit tests were named the
same and the others were accidental duplicates

* Fix F841, remove unused variables

* Fix F821 undefined variables

One of the errors I am leaving in to deal with in a separate PR

* Fix failing unit test

Removed this before because it didn't cause an error on my machine (but
causes it on CI)

* Fix F711 comparison to None

* Fix E731 use def instead of lambda

* Fix F402 overwriting import variable

* Fix E722, bare except

* Fix E721 use is for type comparisons

Run Details

253 of 470 new or added lines in 9 files covered. (53.83%)

67 existing lines in 6 files now uncovered.

1405 of 2660 relevant lines covered (52.82%)

0.53 hits per line