Default to tests running if the skip action errors out

Author: chiedo

.github/workflows/browser-test.yml

@@ -24,17 +24,17 @@ jobs:
     steps:
       # Each of these ifs needs to be repeated at each step to make sure the required check still runs
       # Even if if doesn't do anything
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Checkout
         uses: actions/checkout@a81bbbf8298c0fa03ea29cdc473d45769f953675
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Install
         uses: ianwalter/puppeteer@12728ddef82390d1ecd4732fb543f62177392fbb
         with:
           args: npm ci
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Test
         uses: ianwalter/puppeteer@12728ddef82390d1ecd4732fb543f62177392fbb
         with:

.github/workflows/test.yml

@@ -34,23 +34,23 @@ jobs:
     steps:
       # Each of these ifs needs to be repeated at each step to make sure the required check still runs
       # Even if if doesn't do anything
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Check out repo
         uses: actions/checkout@a81bbbf8298c0fa03ea29cdc473d45769f953675
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Setup node
         uses: actions/setup-node@56899e050abffc08c2b3b61f3ec6a79a9dc3223d
         with:
           node-version: 14.x
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Get npm cache directory
         id: npm-cache
         run: |
           echo "::set-output name=dir::$(npm config get cache)"
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Cache node modules
         uses: actions/cache@d1255ad9362389eac595a9ae406b8e8cb3331f16
         with:
@@ -59,15 +59,15 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-node-
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Install dependencies
         run: npm ci
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Run linter
         run: npx standard
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Check dependencies
         run: npm run check-deps
   test:
@@ -80,23 +80,23 @@ jobs:
     steps:
       # Each of these ifs needs to be repeated at each step to make sure the required check still runs
       # Even if if doesn't do anything
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Check out repo
         uses: actions/checkout@a81bbbf8298c0fa03ea29cdc473d45769f953675
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Setup node
         uses: actions/setup-node@56899e050abffc08c2b3b61f3ec6a79a9dc3223d
         with:
           node-version: 14.x
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Get npm cache directory
         id: npm-cache
         run: |
           echo "::set-output name=dir::$(npm config get cache)"
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Cache node modules
         uses: actions/cache@d1255ad9362389eac595a9ae406b8e8cb3331f16
         with:
@@ -105,15 +105,15 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-node-
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Install dependencies
         run: npm ci
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Run build script
         run: npm run build
 
-      - if: ${{ needs.see_if_should_skip.outputs.should_skip == 'false' }}
+      - if: ${{ needs.see_if_should_skip.outputs.should_skip != 'true' }}
         name: Run tests
         run: npx jest tests/${{ matrix.test-group }}/