Skip to content

Commit c9b2bcb

Browse files
authored
Update benchmarking CI for cargo-codspeed v4 (#20686)
Summary -- A new codspeed [release](https://github.com/CodSpeedHQ/codspeed-rust/releases/tag/v4.0.0) came out, and our CI was failing. It looks like the previously-positional list of benchmarks now corresponds to a `--bench` [flag](https://github.com/CodSpeedHQ/codspeed-rust/blob/7159cf86b9432243565e488c827c1663b5385fde/crates/cargo-codspeed/src/app.rs#L39-L43). Test Plan -- CI on this PR ## TODO - [x] Drop whitespace change commit, just wanted to make sure the benchmarks ran
1 parent 4e94b22 commit c9b2bcb

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/ci.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -932,7 +932,7 @@ jobs:
932932
tool: cargo-codspeed
933933

934934
- name: "Build benchmarks"
935-
run: cargo codspeed build --features "codspeed,instrumented" --no-default-features -p ruff_benchmark formatter lexer linter parser
935+
run: cargo codspeed build --features "codspeed,instrumented" --no-default-features -p ruff_benchmark --bench formatter --bench lexer --bench linter --bench parser
936936

937937
- name: "Run benchmarks"
938938
uses: CodSpeedHQ/action@653fdc30e6c40ffd9739e40c8a0576f4f4523ca1 # v4.0.1
@@ -967,7 +967,7 @@ jobs:
967967
tool: cargo-codspeed
968968

969969
- name: "Build benchmarks"
970-
run: cargo codspeed build --features "codspeed,instrumented" --no-default-features -p ruff_benchmark ty
970+
run: cargo codspeed build --features "codspeed,instrumented" --no-default-features -p ruff_benchmark --bench ty
971971

972972
- name: "Run benchmarks"
973973
uses: CodSpeedHQ/action@653fdc30e6c40ffd9739e40c8a0576f4f4523ca1 # v4.0.1

0 commit comments

Comments
 (0)