Skip to content

fix(types): drop an unused MergedError import in summary_merge_structure - #705

Merged
zzylol merged 1 commit into
mainfrom
fix/unused-approximate-import
Oct 15, 2026
Merged

zzylol merged 1 commit into
mainfrom
fix/unused-approximate-import

Conversation

@zzylol

@zzylol zzylol commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Why

format-and-lint fails on main after #702 (b788fc52): clippy rejects an unused import in crates/types/tests/summary_merge_structure.rs.

What

Drop Approximate from the use MergedError::{…} list; the test refers to it by its qualified name.

How

One-line import change.

Key code interfaces added / modified

None (tests only)

Before this PR

cargo clippy --workspace --all-targets --all-features --locked -- -D warnings fails with unused import: Approximate.

After this PR

Clippy is clean across the workspace.

Verification

  • cargo clippy --workspace --all-targets --all-features --locked -- -D warnings: clean.
  • cargo fmt --all -- --check: clean.
  • No behavior change, so no new tests.

The #696 and #702 merges left `Approximate` imported but used only by its
qualified name, which fails clippy -D warnings on main.
@zzylol
zzylol merged commit bed9b95 into main Oct 15, 2026
3 checks passed
@zzylol zzylol added the bug Something isn't working label Oct 16, 2026
@zzylol
zzylol deleted the fix/unused-approximate-import branch November 3, 2026 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant