-
Notifications
You must be signed in to change notification settings - Fork 45
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #1621 from oasisprotocol/release
Split main workflow into build, lint and test workflows
- Loading branch information
Showing
4 changed files
with
140 additions
and
44 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,37 @@ | ||
# NOTE: This name appears in GitHub's Checks API and in workflow's status badge. | ||
name: ci-build | ||
|
||
# Trigger the workflow when: | ||
on: | ||
# A push occurs to one of the matched branches. | ||
push: | ||
branches: | ||
- master | ||
- stable/* | ||
# Or when a pull request event occurs for a pull request against one of the | ||
# matched branches. | ||
pull_request: | ||
branches: | ||
- master | ||
- stable/* | ||
|
||
# Explicitly disable secrets.GITHUB_TOKEN permissions. | ||
permissions: {} | ||
|
||
jobs: | ||
build: | ||
# NOTE: This name appears in GitHub's Checks API. | ||
name: build | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Checkout code | ||
uses: actions/checkout@v4 | ||
- name: Set up Node.js 18 | ||
uses: actions/setup-node@v3 | ||
with: | ||
node-version: '18.x' | ||
cache: yarn | ||
- name: Install dependencies | ||
run: yarn install --frozen-lockfile | ||
- name: Build app | ||
run: yarn build |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,44 @@ | ||
# NOTE: This name appears in GitHub's Checks API and in workflow's status badge. | ||
name: ci-lint | ||
|
||
# Trigger the workflow when: | ||
on: | ||
# A push occurs to one of the matched branches. | ||
push: | ||
branches: | ||
- master | ||
- stable/* | ||
# Or when a pull request event occurs for a pull request against one of the | ||
# matched branches. | ||
pull_request: | ||
branches: | ||
- master | ||
- stable/* | ||
|
||
# Explicitly disable secrets.GITHUB_TOKEN permissions. | ||
permissions: {} | ||
|
||
jobs: | ||
lint: | ||
# NOTE: This name appears in GitHub's Checks API. | ||
name: lint | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Checkout code | ||
uses: actions/checkout@v4 | ||
- name: Set up Node.js 18 | ||
uses: actions/setup-node@v3 | ||
with: | ||
node-version: '18.x' | ||
cache: yarn | ||
- name: Install dependencies | ||
run: yarn install --frozen-lockfile | ||
- name: ESLint | ||
# Disallow warnings and always throw errors. | ||
run: yarn lint --max-warnings 0 | ||
- name: Validate Grommet icons types | ||
run: | | ||
yarn fix-grommet-icons-types | ||
git diff --exit-code | ||
- name: Validate TypeScript | ||
run: yarn checkTs |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,57 +1,53 @@ | ||
name: Build and test | ||
# NOTE: This name appears in GitHub's Checks API and in workflow's status badge. | ||
name: ci-test | ||
|
||
# Trigger the workflow when: | ||
on: | ||
# A push occurs to one of the matched branches. | ||
push: | ||
branches: [stable, master] | ||
branches: | ||
- master | ||
- stable/* | ||
# Or when a pull request event occurs for a pull request against one of the | ||
# matched branches. | ||
pull_request: | ||
branches: [stable, master] | ||
branches: | ||
- master | ||
- stable/* | ||
|
||
# disable secrets.GITHUB_TOKEN permissions | ||
# Explicitly disable secrets.GITHUB_TOKEN permissions. | ||
permissions: {} | ||
|
||
jobs: | ||
yarn_cache: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Use Node.js 18.x | ||
- name: Checkout code | ||
uses: actions/checkout@v4 | ||
- name: Set up Node.js 18 | ||
uses: actions/setup-node@v3 | ||
id: yarn-cache | ||
with: | ||
node-version: 18 | ||
cache: 'yarn' | ||
cache: yarn | ||
- if: steps.yarn-cache.outputs.cache-hit != 'true' | ||
run: yarn install --frozen-lockfile | ||
|
||
build: | ||
needs: [yarn_cache] | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Use Node.js 18.x | ||
uses: actions/setup-node@v3 | ||
with: | ||
node-version: 18 | ||
cache: 'yarn' | ||
- run: yarn install --frozen-lockfile | ||
- run: yarn lint --max-warnings 0 | ||
- run: | | ||
yarn fix-grommet-icons-types | ||
git diff --exit-code | ||
- run: yarn checkTs | ||
- run: yarn build | ||
|
||
jest: | ||
# NOTE: This name appears in GitHub's Checks API. | ||
name: jest | ||
needs: [yarn_cache] | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Use Node.js 18.x | ||
- name: Checkout code | ||
uses: actions/checkout@v4 | ||
- name: Set up Node.js 18 | ||
uses: actions/setup-node@v3 | ||
with: | ||
node-version: 18 | ||
cache: 'yarn' | ||
- run: yarn install --frozen-lockfile | ||
node-version: '18.x' | ||
cache: yarn | ||
- name: Install dependencies | ||
run: yarn install --frozen-lockfile | ||
- run: yarn test --coverage | ||
- name: 'Upload coverage report' | ||
uses: actions/upload-artifact@v3 | ||
|
@@ -61,16 +57,20 @@ jobs: | |
retention-days: 5 | ||
|
||
playwright: | ||
# NOTE: This name appears in GitHub's Checks API. | ||
name: playwright | ||
needs: [yarn_cache] | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Use Node.js 18.x | ||
- name: Checkout code | ||
uses: actions/checkout@v4 | ||
- name: Set up Node.js 18 | ||
uses: actions/setup-node@v3 | ||
with: | ||
node-version: 18 | ||
cache: 'yarn' | ||
- run: yarn install --frozen-lockfile | ||
node-version: '18.x' | ||
cache: yarn | ||
- name: Install dependencies | ||
run: yarn install --frozen-lockfile | ||
- run: REACT_APP_E2E_TEST=1 yarn build:ext | ||
- run: REACT_APP_E2E_TEST=1 yarn start:prod & | ||
- name: Install playwright's npm dependencies | ||
|
@@ -92,16 +92,20 @@ jobs: | |
retention-days: 5 | ||
|
||
cypress: | ||
# NOTE: This name appears in GitHub's Checks API. | ||
name: cypress | ||
needs: [yarn_cache] | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Use Node.js 18.x | ||
- name: Checkout code | ||
uses: actions/checkout@v4 | ||
- name: Set up Node.js 18 | ||
uses: actions/setup-node@v3 | ||
with: | ||
node-version: 18 | ||
cache: 'yarn' | ||
- run: yarn install --frozen-lockfile | ||
node-version: '18.x' | ||
cache: yarn | ||
- name: Install dependencies | ||
run: yarn install --frozen-lockfile | ||
- run: docker-compose pull | ||
- uses: satackey/[email protected] | ||
# Ignore the failure of a step and avoid terminating the job. | ||
|
@@ -118,10 +122,13 @@ jobs: | |
retention-days: 5 | ||
|
||
upload-coverage: | ||
# NOTE: This name appears in GitHub's Checks API. | ||
name: coverage | ||
needs: [cypress, jest] | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Checkout code | ||
uses: actions/checkout@v4 | ||
- name: Download coverage reports | ||
uses: actions/download-artifact@v3 | ||
- uses: codecov/codecov-action@v3 | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters