Parallel CI jobs #1705
Parallel CI jobs #1705
Conversation
| branches: [master] | ||
| pull_request: | ||
| branches: [master] | ||
| on: push |
There was a problem hiding this comment.
This was previously setup just to run CI on pushes to master, not sure why. It now runs CI on all pushes to this repo.
| - name: 💅🏼 Lint | ||
| run: | | ||
| yarn lint | ||
| yarn ci:lint-docs | ||
|
|
||
| node-build: |
There was a problem hiding this comment.
I wonder if there's a way to re-use all of this duplication :/
There was a problem hiding this comment.
Doesn't seem so - actions/starter-workflows#245.
I stumbled across the needs field while looking through their docs. Couldn't get it to work properly and it arguably ends up creating more duplication - 89013b8.
There was a problem hiding this comment.
|
Hmm, wonder why these are hanging.
|
ismail-syed
force-pushed
the
imporve-ci-setup
branch
3 times, most recently
from
89013b8 to
884056f
Compare
|
Grr we have a flaky test that got flakier after the switch to GitHub Actions somehow. |
|
Actually, it could be a Node 14 perf thing /shrug. Were we running node 14 in Travis?
|
Description
This PR splits up our CI checks into parallel jobs to boost CI times (shaves of a couple mins). This is also helps with CI reporting errors sooner than later since it's parallel.
Before
After