Skip to content

Commit 70a17bf

Browse files
committed
ci: build and test in separate jobs
1 parent 06c7337 commit 70a17bf

File tree

2 files changed

+44
-12
lines changed

2 files changed

+44
-12
lines changed

.github/workflows/CI.yml

+43-10
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,40 @@ on:
66
# - master
77

88
jobs:
9+
Build:
10+
if: "!contains(github.event.head_commit.message, '[skip ci]')"
11+
runs-on: ${{ matrix.os }}
12+
strategy:
13+
fail-fast: false
14+
matrix:
15+
os:
16+
- ubuntu-latest
17+
atom_channel:
18+
- stable
19+
steps:
20+
- uses: actions/checkout@v2
21+
- uses: atom-community/action-setup-atom@v1
22+
with:
23+
channel: ${{ matrix.atom_channel }}
24+
25+
- name: Install dependencies and build
26+
run: |
27+
apm install
28+
29+
npm run clean
30+
npm run tsc || echo done
31+
32+
npm run build
33+
34+
- name: Upload built files
35+
uses: actions/upload-artifact@v2
36+
with:
37+
path: |
38+
./dist
39+
940
Test:
1041
if: "!contains(github.event.head_commit.message, '[skip ci]')"
42+
needs: Build
1143
name: ${{ matrix.os }} - Atom ${{ matrix.atom_channel }}
1244
runs-on: ${{ matrix.os }}
1345
strategy:
@@ -24,11 +56,19 @@ jobs:
2456
with:
2557
channel: ${{ matrix.atom_channel }}
2658

27-
- name: Install dependencies and build
59+
- name: Download articats
60+
uses: actions/download-artifact@v2
61+
- name: Place artifacts
62+
shell: bash
63+
run: |
64+
rm -rf ./dist
65+
mkdir ./dist
66+
cp -r ./artifact/* ./dist
67+
rm -rf ./artifact
68+
69+
- name: Install dependencies
2870
run: |
2971
apm install
30-
npm run tsc || echo done
31-
npm run build
3272
3373
- name: Run tests 👩🏾‍💻
3474
run: npm run test
@@ -55,10 +95,3 @@ jobs:
5595

5696
- name: Lint ✨
5797
run: npm run test.lint
58-
59-
Skip:
60-
if: contains(github.event.head_commit.message, '[skip ci]')
61-
runs-on: ubuntu-latest
62-
steps:
63-
- name: Skip CI 🚫
64-
run: echo skip ci

package.json

+1-2
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,7 @@
4848
"build:services-docs": "markdox lib/services/index.js -o lib/services/README.md",
4949
"build:plugin-docs": "markdox lib/plugin-api/hydrogen-provider.js lib/plugin-api/hydrogen-kernel.js -o docs/PluginAPI.md",
5050
"build:docs": "npm run build:plugin-docs && npm run build:services-docs",
51-
"build-commit": "npm run clean && build-commit -o dist",
52-
"prepare": "npm run build"
51+
"build-commit": "npm run clean && build-commit -o dist"
5352
},
5453
"atomTestRunner": "atom-jasmine3-test-runner",
5554
"repository": "https://github.com/nteract/hydrogen",

0 commit comments

Comments
 (0)