Skip to content

Conversation

@ChrisRackauckas-Claude
Copy link
Contributor

Summary

  • Convert implicit using statements to explicit imports in src/SciMLBenchmarks.jl
  • Add ExplicitImports.jl tests to ensure import hygiene is maintained
  • Add ExplicitImports as a test dependency

The following implicit imports were converted to explicit:

  • IJulia: IJulia
  • InteractiveUtils: InteractiveUtils
  • Markdown: Markdown, @md_str
  • Pkg: Pkg
  • Weave: Weave, tangle, weave

Changes Made

  1. src/SciMLBenchmarks.jl: Changed using Weave, Pkg, IJulia, InteractiveUtils, Markdown to explicit imports
  2. test/explicit_imports.jl: New test file with ExplicitImports checks
  3. test/runtests.jl: Added ExplicitImports test suite
  4. Project.toml: Added ExplicitImports to test dependencies

Test plan

  • Ran ExplicitImports analysis - no implicit imports detected after fix
  • Ran ExplicitImports stale imports check - no stale imports
  • Ran test suite - ExplicitImports tests pass

CC: @ChrisRackauckas

🤖 Generated with Claude Code

- Convert implicit `using` statements to explicit imports in src/SciMLBenchmarks.jl
- Add ExplicitImports.jl tests to ensure import hygiene is maintained
- Add ExplicitImports as a test dependency

The following implicit imports were converted to explicit:
- IJulia: IJulia
- InteractiveUtils: InteractiveUtils
- Markdown: Markdown, @md_str
- Pkg: Pkg
- Weave: Weave, tangle, weave

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

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@ChrisRackauckas ChrisRackauckas merged commit 80385ca into SciML:master Dec 29, 2025
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.

2 participants