Merge pull request #404 from ember-learn/github-ci

simplifying github ci file

Author: mansona

.github/workflows/ci.yml

@@ -4,114 +4,88 @@ on:
   push:
     branches:
       - master
-    tags:
-      - v[0-9]+.[0-9]+.[0-9]+
+      - main
+      - "v*"
   pull_request:
 
 env:
   NODE_VERSION: 10
   PERCY_PARALLEL_NONCE: ${{ github.run_id }}-${{ github.run_number }}
   PERCY_PARALLEL_TOTAL: 1
+  FORCE_COLOR: 1
 
 jobs:
   lint:
-    name: Lint and Test
+    name: Linting
     runs-on: ubuntu-latest
-    timeout-minutes: 3
-    steps:
-      - name: Check out a copy of the repo
-        uses: actions/checkout@v2
-
-      - uses: mansona/npm-lockfile-version@v1
 
-      - name: Use Node.js ${{ env.NODE_VERSION }}
-        uses: actions/setup-node@v2-beta
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-node@v2
         with:
           node-version: ${{ env.NODE_VERSION }}
 
-      - name: Cache npm cache and node_modules
-        id: cache-dependencies
-        uses: actions/cache@v2
-        with:
-          path: |
-            ~/.npm
-            node_modules
-          key: ${{ runner.os }}-${{ env.NODE_VERSION }}-${{ hashFiles('**/package-lock.json') }}
-          restore-keys: ${{ runner.os }}-${{ env.NODE_VERSION }}-
+      - run: npm i -g npm@7
+      - run: npm ci
+      - run: npm run lint
 
-      - name: Install dependencies
-        run: npm install
-        if: steps.cache-dependencies.outputs.cache-hit != 'true'
-
-      - name: Lint
-        run: npm run lint
+  floating-dependencies:
+    name: Floating Dependencies
+    runs-on: ubuntu-latest
 
-      - name: Test
-        uses: percy/exec-action@v0.3.1
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-node@v2
         with:
-          custom-command: npm run test:ember
-        env:
-          PERCY_PARALLEL_NONCE: ${{ env.PERCY_PARALLEL_NONCE }}
-          PERCY_PARALLEL_TOTAL: ${{ env.PERCY_PARALLEL_TOTAL }}
-          PERCY_TOKEN: ee0a9d5c1122d6a21852edf19b5b309aaec18077fb3900c98995c90bc48ed240
+          node-version: ${{ env.NODE_VERSION }}
 
+      - run: npm i -g npm@7
+      - run: npm install --no-package-lock
+      - run: npm test
 
-  test-addon-floating:
-    name: Test addon (floating dependencies)
+  test:
+    name: Tests
     runs-on: ubuntu-latest
-    timeout-minutes: 5
-    steps:
-      - name: Check out a copy of the repo
-        uses: actions/checkout@v2
+    env:
+      PERCY_TOKEN: ee0a9d5c1122d6a21852edf19b5b309aaec18077fb3900c98995c90bc48ed240
+      RUN_PERCY_TESTS: true
+
 
-      - name: Use Node.js ${{ env.NODE_VERSION }}
-        uses: actions/setup-node@v2-beta
+    steps:
+      - uses: actions/checkout@v2
+      - uses: mansona/npm-lockfile-version@v1
+      - uses: actions/setup-node@v2
         with:
           node-version: ${{ env.NODE_VERSION }}
 
-      - name: Install dependencies
-        run: npm install --no-package-lock
-
-      - name: Test
-        run: npm run test:ember
+      - run: npm i -g npm@7
+      - run: npm ci
+      - run: npx percy exec -- npm run test
 
-  test-compatibility:
-    name: Test compatibility
+  try-scenarios:
+    name: "ember-try: ${{ matrix.ember-try-scenario }}"
     runs-on: ubuntu-latest
+    timeout-minutes: 10
+    needs: test
+
     strategy:
-      fail-fast: true
       matrix:
-        scenario:
+        ember-try-scenario:
           - 'ember-lts-3.16'
           - 'ember-lts-3.20'
           - 'ember-release'
           - 'ember-beta'
           - 'ember-canary'
           - 'ember-default-with-jquery'
           - 'ember-classic'
-    timeout-minutes: 7
-    steps:
-      - name: Check out a copy of the repo
-        uses: actions/checkout@v2
 
-      - name: Use Node.js ${{ env.NODE_VERSION }}
-        uses: actions/setup-node@v2-beta
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-node@v2
         with:
           node-version: ${{ env.NODE_VERSION }}
 
-      - name: Cache npm cache and node_modules
-        id: cache-dependencies
-        uses: actions/cache@v2
-        with:
-          path: |
-            ~/.npm
-            node_modules
-          key: ${{ runner.os }}-${{ env.NODE_VERSION }}-${{ matrix.scenario }}-${{ hashFiles('**/package-lock.json') }}
-          restore-keys: ${{ runner.os }}-${{ env.NODE_VERSION }}-${{ matrix.scenario }}-
-
-      - name: Install dependencies
-        run: npm install
-        if: steps.cache-dependencies.outputs.cache-hit != 'true'
-
-      - name: Test
-        run: npx ember try:one $EMBER_TRY_SCENARIO ${{ matrix.scenario }}
+      - run: npm i -g npm@7
+      - run: npm ci
+      - name: test
+        run: node_modules/.bin/ember try:one ${{ matrix.ember-try-scenario }} --skip-cleanup