Skip to content

Commit d8106d3

Browse files
authored
Run codspeed benchmarks with profiling profile (#21261)
## Summary This reduces the walltime benchmarks from 15m to 10m, and we should see an even bigger improvement once build caching kicks in, so I think it's worth the downsides.
1 parent 4fd8d4b commit d8106d3

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

.github/workflows/ci.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -953,7 +953,7 @@ jobs:
953953
tool: cargo-codspeed
954954

955955
- name: "Build benchmarks"
956-
run: cargo codspeed build --features "codspeed,instrumented" --no-default-features -p ruff_benchmark --bench formatter --bench lexer --bench linter --bench parser
956+
run: cargo codspeed build --features "codspeed,instrumented" --profile profiling --no-default-features -p ruff_benchmark --bench formatter --bench lexer --bench linter --bench parser
957957

958958
- name: "Run benchmarks"
959959
uses: CodSpeedHQ/action@6b43a0cd438f6ca5ad26f9ed03ed159ed2df7da9 # v4.1.1
@@ -991,7 +991,7 @@ jobs:
991991
tool: cargo-codspeed
992992

993993
- name: "Build benchmarks"
994-
run: cargo codspeed build --features "codspeed,instrumented" --no-default-features -p ruff_benchmark --bench ty
994+
run: cargo codspeed build --features "codspeed,instrumented" --profile profiling --no-default-features -p ruff_benchmark --bench ty
995995

996996
- name: "Run benchmarks"
997997
uses: CodSpeedHQ/action@6b43a0cd438f6ca5ad26f9ed03ed159ed2df7da9 # v4.1.1
@@ -1029,7 +1029,7 @@ jobs:
10291029
tool: cargo-codspeed
10301030

10311031
- name: "Build benchmarks"
1032-
run: cargo codspeed build --features "codspeed,walltime" --no-default-features -p ruff_benchmark
1032+
run: cargo codspeed build --features "codspeed,walltime" --profile profiling --no-default-features -p ruff_benchmark
10331033

10341034
- name: "Run benchmarks"
10351035
uses: CodSpeedHQ/action@6b43a0cd438f6ca5ad26f9ed03ed159ed2df7da9 # v4.1.1

0 commit comments

Comments
 (0)