From a4d929b19c18b44bb36072717ce1c0e346a5f3ee Mon Sep 17 00:00:00 2001 From: Yii Date: Tue, 17 Jun 2025 13:37:10 +0800 Subject: [PATCH] ci: remove graphite pre check (#12834) ## Summary by CodeRabbit - **Chores** - Updated the CI workflow by removing the "optimize_ci" job and all related dependencies and conditions. All remaining jobs now run unconditionally. --- .github/workflows/build-test.yml | 76 -------------------------------- 1 file changed, 76 deletions(-) diff --git a/.github/workflows/build-test.yml b/.github/workflows/build-test.yml index f9b500718..25685a812 100644 --- a/.github/workflows/build-test.yml +++ b/.github/workflows/build-test.yml @@ -27,26 +27,11 @@ concurrency: cancel-in-progress: true jobs: - optimize_ci: - name: Optimize CI - runs-on: ubuntu-latest - outputs: - skip: ${{ steps.check_skip.outputs.skip }} - steps: - - uses: actions/checkout@v4 - - name: Graphite CI Optimizer - uses: withgraphite/graphite-ci-action@main - id: check_skip - with: - graphite_token: ${{ secrets.GRAPHITE_CI_OPTIMIZER_TOKEN }} - analyze: name: Analyze runs-on: ubuntu-latest env: NODE_OPTIONS: --max-old-space-size=14384 - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' permissions: actions: read contents: read @@ -80,9 +65,6 @@ jobs: lint: name: Lint runs-on: ubuntu-24.04-arm - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' - steps: - uses: actions/checkout@v4 - name: Run oxlint @@ -108,8 +90,6 @@ jobs: typecheck: name: Typecheck runs-on: ubuntu-24.04-arm - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' env: NODE_OPTIONS: --max-old-space-size=14384 steps: @@ -137,8 +117,6 @@ jobs: lint-rust: name: Lint Rust runs-on: ubuntu-latest - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' steps: - uses: actions/checkout@v4 - uses: ./.github/actions/build-rust @@ -159,9 +137,7 @@ jobs: name: Check Git Status runs-on: ubuntu-latest needs: - - optimize_ci - build-server-native - if: needs.optimize_ci.outputs.skip == 'false' steps: - uses: actions/checkout@v4 - name: Setup Node.js @@ -197,8 +173,6 @@ jobs: check-yarn-binary: name: Check yarn binary runs-on: ubuntu-latest - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' steps: - uses: actions/checkout@v4 - name: Run check @@ -209,8 +183,6 @@ jobs: e2e-blocksuite-test: name: E2E BlockSuite Test runs-on: ubuntu-latest - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' strategy: fail-fast: false matrix: @@ -242,8 +214,6 @@ jobs: e2e-blocksuite-cross-browser-test: name: E2E BlockSuite Cross Browser Test runs-on: ubuntu-latest - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' strategy: fail-fast: false matrix: @@ -278,8 +248,6 @@ jobs: e2e-test: name: E2E Test runs-on: ubuntu-24.04-arm - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' env: DISTRIBUTION: web IN_CI_TEST: true @@ -312,8 +280,6 @@ jobs: e2e-mobile-test: name: E2E Mobile Test runs-on: ubuntu-latest - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' env: DISTRIBUTION: mobile IN_CI_TEST: true @@ -345,9 +311,7 @@ jobs: name: Unit Test runs-on: ubuntu-latest needs: - - optimize_ci - build-native - if: needs.optimize_ci.outputs.skip == 'false' env: DISTRIBUTION: web strategy: @@ -384,8 +348,6 @@ jobs: build-native: name: Build AFFiNE native (${{ matrix.spec.target }}) runs-on: ${{ matrix.spec.os }} - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' env: CARGO_PROFILE_RELEASE_DEBUG: '1' strategy: @@ -428,8 +390,6 @@ jobs: build-windows-native: name: Build AFFiNE native (${{ matrix.spec.target }}) runs-on: ${{ matrix.spec.os }} - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' env: CARGO_PROFILE_RELEASE_DEBUG: '1' strategy: @@ -477,8 +437,6 @@ jobs: build-server-native: name: Build Server native runs-on: ubuntu-latest - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' env: CARGO_PROFILE_RELEASE_DEBUG: '1' steps: @@ -504,8 +462,6 @@ jobs: build-electron-renderer: name: Build @affine/electron renderer runs-on: ubuntu-latest - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' steps: - uses: actions/checkout@v4 - name: Setup Node.js @@ -531,9 +487,7 @@ jobs: name: Native Unit Test runs-on: ubuntu-latest needs: - - optimize_ci - build-native - if: needs.optimize_ci.outputs.skip == 'false' steps: - uses: actions/checkout@v4 - name: Setup Node.js @@ -553,9 +507,7 @@ jobs: name: Server Test runs-on: ubuntu-latest needs: - - optimize_ci - build-server-native - if: needs.optimize_ci.outputs.skip == 'false' strategy: fail-fast: false matrix: @@ -628,9 +580,7 @@ jobs: name: Server Test with Elasticsearch runs-on: ubuntu-latest needs: - - optimize_ci - build-server-native - if: needs.optimize_ci.outputs.skip == 'false' strategy: fail-fast: false env: @@ -713,9 +663,7 @@ jobs: name: Server E2E Test runs-on: ubuntu-latest needs: - - optimize_ci - build-server-native - if: needs.optimize_ci.outputs.skip == 'false' env: NODE_ENV: test DATABASE_URL: postgresql://affine:affine@localhost:5432/affine @@ -773,9 +721,6 @@ jobs: miri: name: miri code check runs-on: ubuntu-latest - needs: - - optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' env: RUST_BACKTRACE: full CARGO_TERM_COLOR: always @@ -799,9 +744,6 @@ jobs: loom: name: loom thread test runs-on: ubuntu-latest - needs: - - optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' env: RUSTFLAGS: --cfg loom RUST_BACKTRACE: full @@ -823,9 +765,6 @@ jobs: fuzzing: name: fuzzing runs-on: ubuntu-latest - needs: - - optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' env: CARGO_TERM_COLOR: always steps: @@ -872,9 +811,6 @@ jobs: - { target: 'aarch64-apple-darwin', os: 'macos-latest' } - { target: 'x86_64-pc-windows-msvc', os: 'windows-latest' } - { target: 'aarch64-pc-windows-msvc', os: 'windows-11-arm' } - needs: - - optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' steps: - uses: actions/checkout@v4 - name: Setup Node.js @@ -908,8 +844,6 @@ jobs: rust-test: name: Run native tests runs-on: ubuntu-latest - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' env: CARGO_TERM_COLOR: always steps: @@ -930,9 +864,7 @@ jobs: name: Server Copilot Api Test runs-on: ubuntu-latest needs: - - optimize_ci - build-server-native - if: needs.optimize_ci.outputs.skip == 'false' env: NODE_ENV: test DISTRIBUTION: web @@ -1114,10 +1046,8 @@ jobs: name: ${{ matrix.tests.name }} runs-on: ubuntu-latest needs: - - optimize_ci - build-server-native - build-native - if: needs.optimize_ci.outputs.skip == 'false' env: DISTRIBUTION: web DATABASE_URL: postgresql://affine:affine@localhost:5432/affine @@ -1221,10 +1151,8 @@ jobs: name: Desktop Test (${{ matrix.spec.os }}, ${{ matrix.spec.platform }}, ${{ matrix.spec.arch }}, ${{ matrix.spec.target }}, ${{ matrix.spec.test }}) runs-on: ${{ matrix.spec.os }} needs: - - optimize_ci - build-electron-renderer - build-native - if: needs.optimize_ci.outputs.skip == 'false' strategy: fail-fast: false matrix: @@ -1319,10 +1247,8 @@ jobs: name: Desktop bundle check (${{ matrix.spec.os }}, ${{ matrix.spec.platform }}, ${{ matrix.spec.arch }}, ${{ matrix.spec.target }}, ${{ matrix.spec.test }}) runs-on: ${{ matrix.spec.os }} needs: - - optimize_ci - build-electron-renderer - build-native - if: needs.optimize_ci.outputs.skip == 'false' strategy: fail-fast: false matrix: @@ -1428,8 +1354,6 @@ jobs: test-build-mobile-app: uses: ./.github/workflows/release-mobile.yml - needs: optimize_ci - if: needs.optimize_ci.outputs.skip == 'false' with: build-type: canary build-target: development