Skip to content

Draft: Switch from select to ignore for linting checks

Corné Lukken requested to merge linting-ingore-vs-select into master

Switching from select to ignore has the benefit of intentionally having to disable linting rules instead of accidentally missing important ones.

I have disabled: W291, W293, W391, E201, E201, E202, E203, E221, E222, E225, E226, E231, E241, E251, E252, E261, E301, E302, E303, E305, E306, E401, E402, F403

If you look at the pipeline for this draft you will see quite some important things, unused variables, using variables before it can be guaranteed they are assigned, using a bare except (this will also raise an exception during memory or interrupt exceptions often undesirable), using undefined variables etc.

Merge request reports