Skip to content

Conversation

@ChrisRackauckas-Claude
Copy link

Summary

  • Added ExplicitImports.jl test suite to verify no implicit imports
  • Added ExplicitImports.jl test suite to verify no stale explicit imports
  • Moved ExplicitImports dependency to test dependencies in Project.toml
  • Updated test/runtests.jl to include explicit imports tests

Test Results

All tests pass successfully. The package is currently clean with no import issues detected.

What are ExplicitImports tests?

ExplicitImports.jl tests ensure:

  1. No implicit imports: All names used from dependencies are explicitly imported
  2. No stale imports: No unused imports remain in the code

This prevents issues where code accidentally relies on transitive dependencies or uses names that aren't actually imported.

Test Output

Test Summary:            | Pass  Total  Time
ModelingToolkitCourse.jl |    2      2  2.5s
     Testing ModelingToolkitCourse tests passed

cc: @ChrisRackauckas

🤖 Generated with Claude Code

- Added ExplicitImports.jl test suite to verify no implicit imports
- Added ExplicitImports.jl test suite to verify no stale explicit imports
- Moved ExplicitImports dependency to test dependencies in Project.toml
- Updated test/runtests.jl to include explicit imports tests
- All tests pass successfully

The package is currently clean with no import issues detected.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
@ChrisRackauckas ChrisRackauckas merged commit 2350622 into SciML:main Jan 2, 2026
1 of 2 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.

3 participants