From b73022c81b4ba6c35c2ddd5fe34c23b8e2e1864d Mon Sep 17 00:00:00 2001 From: Oliver Eyton-Williams Date: Thu, 25 Feb 2021 15:36:01 +0100 Subject: [PATCH] fix: make missing scripts fail linting/testing (#41260) If a script is not present, we would like an error letting us know. --- .github/workflows/node.js-tests-upcoming.yml | 6 +++--- .github/workflows/node.js-tests.yml | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/.github/workflows/node.js-tests-upcoming.yml b/.github/workflows/node.js-tests-upcoming.yml index c3252cbceb..cc96954993 100644 --- a/.github/workflows/node.js-tests-upcoming.yml +++ b/.github/workflows/node.js-tests-upcoming.yml @@ -50,8 +50,8 @@ jobs: - name: Lint Source Files run: | npm ci - npm run ensure-env --if-present - npm run lint --if-present + npm run ensure-env + npm run lint test: name: Test @@ -93,7 +93,7 @@ jobs: - name: Install Dependencies run: | npm ci - npm run ensure-env --if-present + npm run ensure-env - name: Run Tests run: npm test diff --git a/.github/workflows/node.js-tests.yml b/.github/workflows/node.js-tests.yml index 8174a88947..bd1be5e768 100644 --- a/.github/workflows/node.js-tests.yml +++ b/.github/workflows/node.js-tests.yml @@ -38,8 +38,8 @@ jobs: - name: Lint Source Files run: | npm ci - npm run ensure-env --if-present - npm run lint --if-present + npm run ensure-env + npm run lint test: name: Test @@ -78,7 +78,7 @@ jobs: - name: Install Dependencies run: | npm ci - npm run ensure-env --if-present + npm run ensure-env - name: Run Tests run: npm test