From 8e05bf28a3f3aa56c18b74603db29915f86617b6 Mon Sep 17 00:00:00 2001 From: Benjamin Heasly Date: Wed, 6 Jul 2022 13:50:07 -0400 Subject: [PATCH] =?UTF-8?q?ci:=20=F0=9F=8E=A1=20Remove=20build=20step=20fr?= =?UTF-8?q?om=20Github=20Actions?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The existing npm/yarn build step runs node with app.js. This fails with the jspsych 7 dependency in place, because jspsych 7 tries to access the browser-only window. Instead of running this step, I think we can skip right to the test phase, and see everything running via jest. --- .github/workflows/test.yml | 3 --- .github/workflows/workflow.yml | 3 --- 2 files changed, 6 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 67bfe81..ab171be 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -20,8 +20,5 @@ jobs: - name: Install dependencies run: yarn install - - name: Build Assets - run: yarn build - - name: Test run: yarn test \ No newline at end of file diff --git a/.github/workflows/workflow.yml b/.github/workflows/workflow.yml index 37b7dde..b0a4b91 100644 --- a/.github/workflows/workflow.yml +++ b/.github/workflows/workflow.yml @@ -21,9 +21,6 @@ jobs: - name: Install dependencies run: yarn install - - name: Build Assets - run: yarn build - - name: Test run: yarn test