Skip to content

fix(ci): remove redundant span test import - #32

Merged
amnn merged 1 commit into
mainfrom
amnn/span-import
Sep 11, 2026
Merged

fix(ci): remove redundant span test import#32
amnn merged 1 commit into
mainfrom
amnn/span-import

Conversation

@amnn

@amnn amnn commented Sep 11, 2026

Copy link
Copy Markdown
Owner

CI fails with warnings denied because the span tests import Stylize both explicitly and through super::*. Remove the redundant test-only import without changing behavior.

Test plan

Reproduced the unused-import failure before the change and confirmed it is resolved afterward. All seven existing app::span::tests tests pass; no new tests are needed for this import-only cleanup.

CI fails with warnings denied because the span tests import `Stylize`
both explicitly and through `super::*`. Remove the redundant test-only
import without changing behavior.

## Test plan

Reproduced the unused-import failure before the change and confirmed it
is resolved afterward. All seven existing `app::span::tests` tests pass;
no new tests are needed for this import-only cleanup.
@amnn amnn self-assigned this Sep 11, 2026
@amnn
amnn marked this pull request as ready for review September 11, 2026 12:24
@amnn
amnn merged commit 4a93481 into main Sep 11, 2026
4 checks passed
@amnn
amnn deleted the amnn/span-import branch September 11, 2026 12:24
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