From 9aac2a44ed6e9ea62701c979a414f5b44ce68017 Mon Sep 17 00:00:00 2001 From: Landon Grindheim Date: Tue, 30 Aug 2022 14:32:06 +0000 Subject: [PATCH] Build the `dist/` directory for dev depenencies We've had several builds fail because the `dist/` directory has changes in it after it builds. I don't know the history of excluding dev dependencies from this check, but it seems to be wrong. --- .github/workflows/dependabot-build.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/dependabot-build.yml b/.github/workflows/dependabot-build.yml index 5aea933..4c5598c 100644 --- a/.github/workflows/dependabot-build.yml +++ b/.github/workflows/dependabot-build.yml @@ -26,8 +26,8 @@ jobs: build-dependabot-changes: runs-on: ubuntu-latest needs: [fetch-dependabot-metadata] - # We only need to build the dist/ folder if the PR relates to Docker or a production npm dependency, otherwise we don't expect changes. - if: needs.fetch-dependabot-metadata.outputs.package-ecosystem == 'docker' || ( needs.fetch-dependabot-metadata.outputs.package-ecosystem == 'npm_and_yarn' && needs.fetch-dependabot-metadata.outputs.dependency-type == 'direct:production' ) + # We only need to build the dist/ folder if the PR relates to Docker or an npm dependency + if: needs.fetch-dependabot-metadata.outputs.package-ecosystem == 'docker' || needs.fetch-dependabot-metadata.outputs.package-ecosystem == 'npm_and_yarn' steps: # Check out using a PAT so any pushed changes will trigger checkruns - uses: actions/checkout@v3