Skip to content

fix(eval): reflow evaluators_safety_test imports to satisfy rustfmt#825

Merged
jwesleye merged 2 commits intointegrationfrom
fix/fmt-safety-test
Apr 23, 2026
Merged

fix(eval): reflow evaluators_safety_test imports to satisfy rustfmt#825
jwesleye merged 2 commits intointegrationfrom
fix/fmt-safety-test

Conversation

@jwesleye
Copy link
Copy Markdown
Collaborator

Summary

Tiny follow-up: integration's `cargo fmt --check` fails on `eval/tests/evaluators_safety_test.rs:12` because rustfmt reflows the import block differently. This PR runs cargo fmt and pushes the fix.

Cause

The import block arrived via PR #821's US1c merge + reconciliation commit 6776a9a. I added `Aggregator` in a companion rescue path (PR #823 branch) without re-running fmt on the file.

Test plan

  • `cargo fmt --check -p swink-agent-eval` clean

🤖 Generated with Claude Code

PR #823's rescue commit (and its companion reconciliation 6776a9a)
landed an import block that rustfmt reflows differently. cargo fmt
--check on the post-merge integration tip caught it.
@jwesleye jwesleye requested a review from unseriousAI as a code owner April 23, 2026 22:48
@github-actions
Copy link
Copy Markdown

Benchmark comparison

group                                    base                                   pr
-----                                    ----                                   --
compact_sliding_window/heavy_500_msgs    1.00     84.7±1.80µs        ? ?/sec    1.00     84.5±1.88µs        ? ?/sec
compact_sliding_window/no_op_100_msgs    1.01      2.6±0.12µs        ? ?/sec    1.00      2.6±0.07µs        ? ?/sec

@jwesleye jwesleye merged commit d077b8b into integration Apr 23, 2026
5 of 8 checks passed
@jwesleye jwesleye deleted the fix/fmt-safety-test branch April 23, 2026 23:05
@github-actions
Copy link
Copy Markdown

Benchmark comparison

group                                    base                                   pr
-----                                    ----                                   --
compact_sliding_window/heavy_500_msgs    1.03     91.5±0.67µs        ? ?/sec    1.00     88.9±0.91µs        ? ?/sec
compact_sliding_window/no_op_100_msgs    1.00      2.7±0.08µs        ? ?/sec    1.02      2.7±0.15µs        ? ?/sec

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant