From 16cdf94001001020762f79f6eca971e26be8dd9e Mon Sep 17 00:00:00 2001
From: Penar Musaraj <pmusaraj@gmail.com>
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 d085046ad6cf9e..1ac4f9c87f0acb 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: