From ad5ca1d03eeecc9de9e1aec0492088c2ec2cb9ff Mon Sep 17 00:00:00 2001 From: "gaoyuan.1226" Date: Mon, 14 Oct 2024 11:45:48 +0800 Subject: [PATCH] fix: test comment --- .github/actions/build-rsbuild/action.yaml | 4 ++-- .github/workflows/ecosystem-ci-from-commit.yml | 15 +++++++++------ tests/plugins.ts | 10 +++++----- 3 files changed, 16 insertions(+), 13 deletions(-) diff --git a/.github/actions/build-rsbuild/action.yaml b/.github/actions/build-rsbuild/action.yaml index e7c43a9..ecd52d4 100644 --- a/.github/actions/build-rsbuild/action.yaml +++ b/.github/actions/build-rsbuild/action.yaml @@ -16,7 +16,7 @@ inputs: node-version: description: 'The version of Node.js to set up' required: true - default: '18' + default: '20' outputs: artifact-name: description: 'The name of the uploaded artifact' @@ -25,7 +25,7 @@ runs: steps: - uses: actions/setup-node@v3 with: - node-version: 18 + node-version: 20 - shell: bash name: Install package manager run: | diff --git a/.github/workflows/ecosystem-ci-from-commit.yml b/.github/workflows/ecosystem-ci-from-commit.yml index 87117e2..78aec94 100644 --- a/.github/workflows/ecosystem-ci-from-commit.yml +++ b/.github/workflows/ecosystem-ci-from-commit.yml @@ -69,14 +69,14 @@ jobs: strategy: matrix: include: - - suite: modernjs - os: ubuntu-latest + # - suite: modernjs + # os: ubuntu-latest - suite: plugins os: ubuntu-latest - - suite: rspress - os: ubuntu-latest - - suite: rslib - os: ubuntu-latest + # - suite: rspress + # os: ubuntu-latest + # - suite: rslib + # os: ubuntu-latest - suite: examples os: ubuntu-latest fail-fast: false @@ -150,3 +150,6 @@ jobs: repo: 'rsbuild', body }) + + console.log(`https://github.com/web-infra-dev/rsbuild/commit/${context.payload.inputs.commitSHA}#comments`) + return `https://github.com/web-infra-dev/rsbuild/commit/${context.payload.inputs.commitSHA}#comments` diff --git a/tests/plugins.ts b/tests/plugins.ts index c56c90f..70579a1 100644 --- a/tests/plugins.ts +++ b/tests/plugins.ts @@ -77,11 +77,11 @@ export async function test(options: RunOptions) { test: [ 'build', async () => { - // if (hasTestScript) { - await $`pnpm run test` - // } else { - // console.warn(`not found test script in ${repo}`) - // } + if (hasTestScript) { + await $`pnpm run test` + } else { + console.warn(`not found test script in ${repo}`) + } }, ], }).catch((err) => {