remove safari test, update snapshots for new macOS, use dependency caching (#745)

* remove safari test

* add node modules caching

* update macos snapshots

* try to fix caching

* try like this

* cache .npm

* adjust cache keys
This commit is contained in:
Konstantin Vyatkin 2019-11-08 03:48:29 -04:00 committed by Josh Johnson
parent b5b593a62f
commit 7dcc155b8f
3 changed files with 19 additions and 2 deletions

Binary file not shown.

Before

Width:  |  Height:  |  Size: 48 KiB

After

Width:  |  Height:  |  Size: 44 KiB

BIN
.github/actions-scripts/__snapshots__/puppeteer-darwin.png vendored Executable file → Normal file

Binary file not shown.

Before

Width:  |  Height:  |  Size: 264 KiB

After

Width:  |  Height:  |  Size: 268 KiB

View file

@ -28,6 +28,8 @@ jobs:
browser: ie
- os: macOS-latest
browser: chrome
# Safari workaround is not working in Catalina
- browser: safari
runs-on: ${{ matrix.os }}
steps:
@ -42,6 +44,17 @@ jobs:
- uses: actions/setup-node@v1
with:
node-version: '12.x'
- name: Cache node modules
uses: actions/cache@v1
with:
path: ~/.npm
key: ${{ runner.OS }}-build-${{ matrix.browser }}
restore-keys: |
${{ runner.OS }}-build-${{ env.cache-name }}-
${{ runner.OS }}-build-
${{ runner.OS }}-
- run: |
npm ci
npm run build
@ -94,9 +107,13 @@ jobs:
- uses: actions/checkout@v1
with:
fetch-depth: 1
- uses: actions/setup-node@v1
- name: Cache node modules
uses: actions/cache@v1
with:
node-version: '12.x'
path: ~/.npm
key: ${{ runner.OS }}-build-puppeteer
restore-keys: |
${{ runner.OS }}-build-puppeteer
- run: |
npm ci
npm run build