(S)CSS Linting
When running the ebi-framework we are greeted with several linting errors.
Adding linting makes development easier as it catches human errors. It also keeps developers adhering to any CSS guidelines decided.
Do we want to stick with the default linting errors? From looking at the output when running a lint task on the current codebase we have errors for:
-
#ID
selectors being used -
!important
being used - nesting levels of Sass.
Linting gives the opportunity to fail builds, possibly fail PRs with CI/CD - should we look to do this?
Should these be a 'notification' only for developers on the project with someone fixing it for them so they're not bogged down by the rules?
Or, should we use something like Prettier to auto format the CSS so it saves time?