You need to sign in or sign up before continuing.
Add back linting checks to CI
Clarification and motivation
During !931 (merged) the trailing whitespaces check was moved from a bash script to a danger one.
However, in doing so, we accidentally also removed the hlint
and the reuse lint
check (see 05dd601b)
These aren't covered by danger, so they should be put back.
NOTE: after this and #674 (closed) we'll have a very high coverage of the style guide covered by CI, however it's not 100% and there are still useful information in there, so IMO we should keep the related checkbox in the MR template.
Acceptance criteria
Both hlint
and reuse lint
are run in CI as part of MR validation.