Skip to content

Use ExplicitImports in tests and fix import issues#4155

Merged
schillic merged 3 commits intomasterfrom
schillic/ExplicitImports
Jan 15, 2026
Merged

Use ExplicitImports in tests and fix import issues#4155
schillic merged 3 commits intomasterfrom
schillic/ExplicitImports

Conversation

@schillic
Copy link
Copy Markdown
Member

@schillic schillic commented Jan 10, 2026

This PR adds ExplicitImports to the tests. It also fixes most issues reported by ExplicitImports such as:

  • removing stale imports
  • explicitly importing symbols from other modules (instead of importing all symbols)

@schillic schillic force-pushed the schillic/ExplicitImports branch 6 times, most recently from 981bb45 to e81ee0a Compare January 11, 2026 12:13
@schillic schillic changed the title Use ExplicitImports in tests and fix most import issues Use ExplicitImports in tests and fix import issues Jan 11, 2026
@schillic schillic force-pushed the schillic/ExplicitImports branch 3 times, most recently from ae82244 to 4319f4a Compare January 11, 2026 13:54
@schillic schillic marked this pull request as ready for review January 11, 2026 14:28
@schillic schillic force-pushed the schillic/ExplicitImports branch from 4319f4a to 30239c1 Compare January 15, 2026 20:07
@schillic schillic force-pushed the schillic/ExplicitImports branch from 30239c1 to a4ba17b Compare January 15, 2026 20:52
@schillic schillic merged commit 7d80174 into master Jan 15, 2026
6 checks passed
@schillic schillic deleted the schillic/ExplicitImports branch January 15, 2026 21:16
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