From 31554496650993d6a3d34626279dcd6f26640a61 Mon Sep 17 00:00:00 2001 From: Adam Perry Date: Fri, 5 Jun 2026 18:04:10 -0700 Subject: [PATCH] Remove `just bench-smoke` from `just test`. --- AGENTS.md | 6 ++++++ justfile | 2 -- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index 4666566f9603..46d2626e321d 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -192,6 +192,12 @@ If you don’t have the tool: - `cargo install --locked cargo-insta` +### Benchmarks + +cargo benchmarks can be run with `just bench`, use the divan crate to write new ones. + +Use `just bench-smoke` to dry-run the benchmark for a single iteration to ensure it works. + ### Test assertions - Tests should use pretty_assertions::assert_eq for clearer diffs. Import this at the top of the test module if it isn't already. diff --git a/justfile b/justfile index fe7e7349b314..3b993b767d7d 100644 --- a/justfile +++ b/justfile @@ -76,12 +76,10 @@ install: [unix] test *args: RUST_MIN_STACK={{ rust_min_stack }} cargo nextest run --no-fail-fast "$@" - just bench-smoke [windows] test *args: $env:RUST_MIN_STACK = "{{ rust_min_stack }}"; cargo nextest run --no-fail-fast @($args | Select-Object -Skip 1) - just bench-smoke # Run from the repository root so scripts that resolve paths from `cwd` see # the same layout they use in GitHub Actions.