Skip to content

Commit 6117e08

Browse files
Boshenclaude
andcommitted
ci: remove detect-changes job
Replace detect-changes dependency with optimize-ci directly. The paths-filter was not working reliably and the workflow-level paths-ignore already handles markdown exclusion for push events. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 58b7ae7 commit 6117e08

1 file changed

Lines changed: 5 additions & 23 deletions

File tree

.github/workflows/ci.yml

Lines changed: 5 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -34,27 +34,9 @@ jobs:
3434
with:
3535
graphite_token: ${{ secrets.GRAPHITE_CI_OPTIMIZER_TOKEN }}
3636

37-
detect-changes:
38-
runs-on: ubuntu-latest
37+
download-previous-rolldown-binaries:
3938
needs: optimize-ci
4039
if: needs.optimize-ci.outputs.skip == 'false'
41-
permissions:
42-
contents: read
43-
pull-requests: read
44-
outputs:
45-
code-changed: ${{ steps.filter.outputs.code }}
46-
steps:
47-
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
48-
- uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3.0.2
49-
id: filter
50-
with:
51-
filters: |
52-
code:
53-
- '**'
54-
55-
download-previous-rolldown-binaries:
56-
needs: detect-changes
57-
if: needs.detect-changes.outputs.code-changed == 'true'
5840
runs-on: ubuntu-latest
5941
permissions:
6042
contents: read
@@ -65,9 +47,9 @@ jobs:
6547
with:
6648
github-token: ${{ secrets.GITHUB_TOKEN }}
6749
test:
68-
needs: detect-changes
50+
needs: optimize-ci
6951
if: |
70-
needs.detect-changes.outputs.code-changed == 'true' && (
52+
needs.optimize-ci.outputs.skip == 'false' && (
7153
matrix.os == 'ubuntu-latest' ||
7254
github.event_name != 'pull_request' ||
7355
contains(github.event.pull_request.labels.*.name, 'ci: full')
@@ -116,8 +98,8 @@ jobs:
11698
- run: cargo test $(for d in crates/*/; do echo -n "-p $(basename $d) "; done) -p vite-plus-cli
11799

118100
lint:
119-
needs: detect-changes
120-
if: needs.detect-changes.outputs.code-changed == 'true'
101+
needs: optimize-ci
102+
if: needs.optimize-ci.outputs.skip == 'false'
121103
name: Lint
122104
runs-on: ubuntu-latest
123105
steps:

0 commit comments

Comments
 (0)