From 488b8b369ab567d887f492e4b07a8c7c4dba92d2 Mon Sep 17 00:00:00 2001 From: David Taylor Date: Thu, 2 Feb 2023 12:34:39 +0000 Subject: [PATCH] DEV: Fix syntax_tree in GitHub CI This broke because of directory ownership errors during `git ls-files`. This commit fixes the permissions and adds bash flags so that those kind of errors will blow up the step in future. --- .github/workflows/linting.yml | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/.github/workflows/linting.yml b/.github/workflows/linting.yml index 984a9d104b7..6489153036d 100644 --- a/.github/workflows/linting.yml +++ b/.github/workflows/linting.yml @@ -22,6 +22,9 @@ jobs: timeout-minutes: 30 steps: + - name: Set working directory owner + run: chown root:root . + - uses: actions/checkout@v3 with: fetch-depth: 1 @@ -70,7 +73,9 @@ jobs: - name: syntax_tree if: ${{ !cancelled() }} - run: bundle exec stree check Gemfile $(git ls-files '*.rb') $(git ls-files '*.rake') + run: | + set -E + bundle exec stree check Gemfile $(git ls-files '*.rb') $(git ls-files '*.rake') - name: ESLint (core) if: ${{ !cancelled() }}