Skip to content

perf: avoid unused legacy serialization and unit formatting - #153

Merged
maebli merged 1 commit into
mainfrom
perf/lazy-rendering
Sep 27, 2026
Merged

maebli merged 1 commit into
mainfrom
perf/lazy-rendering

Conversation

@maebli

@maebli maebli commented Sep 27, 2026

Copy link
Copy Markdown
Owner

Legacy JSON and YAML rendering now serialize only the requested format; wide CSV and summary-only consumers skip both serializations. Canonical unit formatting generates its display fallback only when UCUM conversion fails, avoiding unused formatting and a metadata traversal when conversion succeeds.

Validation:

  • All 700 captured outputs across 100 fixtures and seven formats match byte-for-byte.
  • cargo test --all-features and cargo test --features std pass.
  • Library Clippy with all features, formatting, and diff checks pass.
  • Added coverage for mixed supported/unsupported units and empty units.

Root all-target Clippy remains blocked by existing warnings in unrelated annotation and integration tests.

@maebli
maebli merged commit b372073 into main Sep 27, 2026
9 checks passed
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