From df6003aef50c2640f8fb73b9e9cf2c06f3afd9f7 Mon Sep 17 00:00:00 2001 From: Martin Brennan Date: Mon, 22 Nov 2021 11:30:12 +1000 Subject: [PATCH] DEV: Lint SCSS with prettier in pre-commit (#15033) We are linting SCSS on the GitHub actions CI but not on pre-commit, which can lead to lint failures in CI. Better to warn developers about this locally like our other lints. --- lefthook.yml | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/lefthook.yml b/lefthook.yml index 534e33e87b..41978efbf7 100644 --- a/lefthook.yml +++ b/lefthook.yml @@ -8,6 +8,10 @@ pre-commit: glob: "*.js" include: "app/assets/javascripts|test/javascripts" run: yarn pprettier --list-different {staged_files} + prettier-scss: + glob: "*.scss" + include: "app/assets/stylesheets" + run: yarn pprettier --list-different {staged_files} eslint: glob: "*.js" include: "app/assets/javascripts|test/javascripts" @@ -51,6 +55,10 @@ lints: glob: "*.js" include: "app/assets/javascripts|test/javascripts" run: yarn pprettier --list-different {all_files} + prettier-scss: + glob: "*.scss" + include: "app/assets/stylesheets" + run: yarn pprettier --list-different {all_files} eslint-assets-js: run: yarn eslint app/assets/javascripts eslint-test-js: