Fail pipelines which fail prettier code formatting standards
We have not been enforcing our code formatting standards with prettier largely because over a third of our codebase has not yet been updated to that standard.
Let's "prettify" the remainder of our frontend codebase and then make this a hard failure in CI once we're done.
Steps
- Prettify all CE-only javascript modules which won't cause conflicts with EE in large batches
- Prettify all EE-only javascript modules in large batches
- Prettify remaining files (which have EE-specific changes) one at a time or in small groups to reduce burden on reviewer
- Make a decision regarding prettify standards and our SCSS stylesheets and either prettify them or exclude them from our tooling
- Ensure running
yarn prettier-all
no longer produces any changes in the repo (e.g. we've reached 100%) - Enable a check for
prettier
violations within thestatic-analysis
job in CI