From 16cdf94001001020762f79f6eca971e26be8dd9e Mon Sep 17 00:00:00 2001 From: Penar Musaraj Date: Fri, 15 Nov 2019 09:21:51 -0500 Subject: [PATCH] DEV: Fix files targetted by Lefthook Prettier The previous glob rule was skipping files in sub-subfolders, causing discrepancies between Travis and Jenkins results. --- lefthook.yml | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/lefthook.yml b/lefthook.yml index d085046ad6c..1ac4f9c87f0 100644 --- a/lefthook.yml +++ b/lefthook.yml @@ -2,23 +2,23 @@ pre-commit: parallel: true commands: rubocop: - glob: '*.rb' + glob: "*.rb" run: bundle exec rubocop {staged_files} eslint: - glob: '*.{js,es6}' + glob: "*.{js,es6}" run: yarn eslint --ext .es6 -f compact {staged_files} yaml-syntax: - glob: '*.{yaml,yml}' + glob: "*.{yaml,yml}" run: bundle exec yaml-lint {staged_files} commands: &commands bundle-install: files: git diff --name-only HEAD master - glob: '{Gemfile,Gemfile.lock,*.gemspec}' + glob: "{Gemfile,Gemfile.lock,*.gemspec}" run: bundle install yarn-install: files: git diff --name-only HEAD master - glob: '{package.json,yarn.lock}' + glob: "{package.json,yarn.lock}" run: yarn install post-checkout: @@ -36,7 +36,8 @@ lints: rubocop: run: bundle exec rubocop --parallel prettier: - run: yarn prettier --list-different app/assets/stylesheets/**/*.scss app/assets/javascripts/**/*.es6 test/javascripts/**/*.es6 + glob: "{app/assets/stylesheets/**/*.scss,app/assets/javascripts/**/*.es6,test/javascripts/**/*.es6}" + run: yarn prettier --list-different {all_files} eslint-assets: run: yarn eslint --ext .es6 app/assets/javascripts eslint-test: