Gate the tracing label behind the tracing feature

Closed
JE
2mo ago
+6-10

Main has been red on Rust Tests since #165 merged: cargo clippy --no-default-features -D warnings fails on trace::label and trace::is_raw being dead and the label import in client.rs unused. Without the tracing feature the debug! macro expands to nothing, so the label(operation) in its arguments is never referenced.

#155 added that clippy step and #165 added the spans. Each was green on its own; #165's CI ran before #155 landed and it merged without a re-run.

The fix gates label behind the feature, folds the one-line is_raw helper into it so the OperationInfo import can go, and names label fully qualified at the three call sites the way they already name the macro, so the import line needs no cfg.

make rs-check rs-check-drift is green locally. #171 (the zizmor-action bump) inherits this failure through its merge commit and will need a rebase once this lands.

Summary by cubic

Fixes the cargo clippy --no-default-features -D warnings failure on main by gating the tracing label behind the tracing feature.

  • Folds the is_raw helper into label so the unused OperationInfo import goes away.
  • Names label fully qualified at call sites so the import line needs no cfg.