style: apply rustfmt to the BES tracing sink - #1401
Conversation
The format CI task flagged `crates/axl-runtime/src/engine/bazel/sink/tracing.rs`: the two `TestStatus::try_from` bindings wrapped on the wrong side of the call, the multi-argument `assert_eq!`s exceeded the line width, and the `Timestamp` struct literal needed expanding. Formatting only — no behavior change.
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
|
73f9f29
into
claude/axl-cli-otel-test-spans-52xxpx
|
Closing as redundant — the rustfmt commit (73f9f29) was pushed directly onto This PR could never have been verified on its own: Generated by Claude Code |
Fixes the
format-gha/format-gha-debug/format-format-repeat-taskfailures on #1369. The format task reported exactly one unformatted file,crates/axl-runtime/src/engine/bazel/sink/tracing.rs:TestStatus::try_from(...).unwrap_or(...)bindings wrapped on the wrong side of the callassert_eq!s in the new tests exceeded the line widthTimestamp { seconds: 42, nanos: 0 }literal needed expandingProduced by running
cargo fmt --all;cargo fmt --all -- --checkis now clean across the workspace.Based on
claude/axl-cli-otel-test-spans-52xxpxso the diff is only the formatting change. Merging it into that branch turns #1369's format checks green.Changes are visible to end-users: no
Whitespace and line wrapping only — no behavior change.
Test plan
cargo fmt --all -- --checkpasses with no diffcargo test -p axl-runtime --lib engine::bazel::sink::tracing— 6 passed, 0 failedGenerated by Claude Code