From 5326d9f751756d4289349d7d90ecb4054c2eb2da Mon Sep 17 00:00:00 2001 From: Tony Date: Sat, 5 Feb 2022 15:36:14 -0200 Subject: [PATCH] chore(misc): multiple enhancement to github actions (#9000) * chore: test workflow use redis * chore: docker build cache sharing * chore: eslint format yml files * chore: nodemon restart on art template files changes * chore: run test on multi node and os * chore: unnecessary single version matrix build * chore: fix container operations Container operations are only supported on Linux runners * chore: fix gha syntax * chore: fix node version not show up * revert: "chore: test workflow use redis" This reverts commit df818fd697b4c9151ac635cdb5918e03a4b37062. * revert: "chore: fix container operations" This reverts commit c949368aa7048b9d0f3f1942501c695f10f07f06. * revert: "chore: run test on multi node and os" This reverts commit 6b1cfde579095e73d0314346fb1a2d12d0de00ac. * chore: run test on multi node version --- .github/workflows/build-assets.yml | 11 +++-------- .github/workflows/docker-release.yml | 4 ++-- .github/workflows/docker-test.yml | 6 ++++-- .github/workflows/format.yml | 2 +- .github/workflows/test.yml | 3 ++- package.json | 9 ++++++--- 6 files changed, 18 insertions(+), 17 deletions(-) diff --git a/.github/workflows/build-assets.yml b/.github/workflows/build-assets.yml index 8ad36cd20..388d60618 100644 --- a/.github/workflows/build-assets.yml +++ b/.github/workflows/build-assets.yml @@ -11,20 +11,15 @@ jobs: build: runs-on: ubuntu-latest - strategy: - matrix: - node-version: - - 14.x - steps: - name: Checkout uses: actions/checkout@v2 - - name: Use Node.js ${{ matrix.node-version }} + - name: Use Node.js v14 uses: actions/setup-node@v2 with: - node-version: ${{ matrix.node-version }} + node-version: '14' cache: 'yarn' - - name: build file + - name: Build file run: yarn && npm run build:all - name: Deploy uses: peaceiris/actions-gh-pages@v3 diff --git a/.github/workflows/docker-release.yml b/.github/workflows/docker-release.yml index 8f3a77e0f..ad05e4ea8 100644 --- a/.github/workflows/docker-release.yml +++ b/.github/workflows/docker-release.yml @@ -48,5 +48,5 @@ jobs: tags: ${{ steps.meta.outputs.tags }} labels: ${{ steps.meta.outputs.labels }} platforms: linux/amd64,linux/arm/v7,linux/arm64 - cache-from: type=gha,scope=${{ github.workflow }} - cache-to: type=gha,mode=max,scope=${{ github.workflow }} + cache-from: type=gha,scope=docker-release + cache-to: type=gha,mode=max,scope=docker-release diff --git a/.github/workflows/docker-test.yml b/.github/workflows/docker-test.yml index 7ce30af77..0cf36aca5 100644 --- a/.github/workflows/docker-test.yml +++ b/.github/workflows/docker-test.yml @@ -38,8 +38,10 @@ jobs: load: true tags: ${{ steps.meta.outputs.tags }} labels: ${{ steps.meta.outputs.labels }} - cache-from: type=gha,scope=${{ github.workflow }} - cache-to: type=gha,mode=max,scope=${{ github.workflow }} + cache-from: | + type=gha,scope=docker-test + type=gha,scope=docker-release + cache-to: type=gha,mode=max,scope=docker-test - name: Run dockerfile run: | diff --git a/.github/workflows/format.yml b/.github/workflows/format.yml index e44f5f39f..141cf13fb 100644 --- a/.github/workflows/format.yml +++ b/.github/workflows/format.yml @@ -6,7 +6,7 @@ on: - master jobs: - build: + format: runs-on: ubuntu-latest steps: diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 9412120fd..61fae65ac 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -15,8 +15,9 @@ jobs: strategy: matrix: - node-version: ['14.x'] + node-version: ['14', '16'] + name: Node ${{ matrix.node-version }} steps: - uses: actions/checkout@v2 - uses: actions/setup-node@v2 diff --git a/package.json b/package.json index 5768e5906..b8c20d140 100644 --- a/package.json +++ b/package.json @@ -16,9 +16,9 @@ "build:radar": "node scripts/workflow/build-radar.js", "build:maintainer": "node scripts/workflow/build-maintainer.js", "lint": "eslint .", - "format": "eslint \"**/*.js\" --fix && node docs/.format/format.js && prettier \"**/*.{js,json}\" --write", - "format:staged": "eslint \"**/*.js\" --fix && node docs/.format/format.js --staged && pretty-quick --staged --verbose --pattern \"**/*.{js,json}\"", - "format:check": "eslint \"**/*.js\" && prettier-check \"**/*.{js,json}\"", + "format": "eslint \"**/*.{js,yml}\" --fix && node docs/.format/format.js && prettier \"**/*.{js,json}\" --write", + "format:staged": "eslint \"**/*.{js,yml}\" --fix && node docs/.format/format.js --staged && pretty-quick --staged --verbose --pattern \"**/*.{js,json}\"", + "format:check": "eslint \"**/*.{js,yml}\" && prettier-check \"**/*.{js,json}\"", "test": "npm run format:check && cross-env NODE_ENV=test jest --coverage --runInBand --forceExit", "jest": "cross-env NODE_ENV=test jest --runInBand --forceExit", "jest:watch": "cross-env NODE_ENV=test jest --watch" @@ -152,5 +152,8 @@ "moduleNameMapper": { "@/(.*)": "/lib/$1" } + }, + "nodemonConfig": { + "ext": "art,js,mjs,json" } }