Add --test flag for app#166
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds a --test flag to the ExplicitImports command-line application, enabling users to run Test.jl-style checks instead of printing results. This complements the existing --check and --print functionality.
Changes:
- Added
--testflag to run Test.jl-style checks viatest_explicit_imports() - Implemented mutual exclusivity between
--testand--check/--checklistflags - Updated help text and README documentation to describe the new flag
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/main.jl | Added --test flag parsing, mutual exclusivity check, and execution logic for running test-style checks |
| test/main.jl | Added comprehensive tests for the new --test flag, including tests for standalone usage and mutual exclusivity with other flags |
| README.md | Updated documentation to include the new --test option in the OPTIONS section |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| printstyled(stderr, "ERROR: "; bold=true, color=:red) | ||
| Base.showerror(stderr, e) |
There was a problem hiding this comment.
Consider adding a newline after the error output for better formatting. This would ensure the error message is properly terminated regardless of whether the exception's error formatting includes a trailing newline. This same pattern exists in other error handling blocks in this file (lines 108-109, 244-245), so for consistency, all instances should be updated together if this change is made.
…es (#165) * expand `ignore` functionality to consistently allow ignoring submodules of the package * validate ignores & tweak wording * add tests for validation * tweak wording * add top-level ignore to `test_` fn * rm unnecessary fn * rm another unused fn
to use new test functionality with #143