This PR does first 50% of #1211, that is: replace very_good_analysis with leancode_lint and fix all warnings that arose (~90 - not so bad, I fixed them all).
The last 50% (integration of custom_lint + fixing warnings that will be triggered) I plan to do in a follow-up PR to make review easier. But if you prefer to have it in a single PR, then that's cool as well.
Review checklist
[x] Terms and conditions reflect the current change
[x] Contains enough appropriate tests
[ ] If aimed at production, writes a new summary in whatsnew/whatsnew-pt-PT
[ ] Properly adds an entry in changelog.md with the change
[ ] If PR includes UI updates/additions, its description has screenshots
This PR does first 50% of #1211, that is: replace
very_good_analysis
withleancode_lint
and fix all warnings that arose (~90 - not so bad, I fixed them all).The last 50% (integration of
custom_lint
+ fixing warnings that will be triggered) I plan to do in a follow-up PR to make review easier. But if you prefer to have it in a single PR, then that's cool as well.Review checklist
whatsnew/whatsnew-pt-PT
changelog.md
with the change