Skip to content

Conversation

@ericphanson
Copy link
Member

Since we already have an @main this is all that is needed for 1.12's app support. The only question is what to name the CLI app; here I've chosen explicit-imports.

@DilumAluthge
Copy link
Member

Is this the name that will be put in the PATH? If so, maybe julia-explicit-imports or explicit-imports-julia would be more explicit?

@ericphanson
Copy link
Member Author

Yeah, it is the name that goes in the PATH. I think those could make sense. Or jl-explicit-imports or something. I wonder if there should be some convention?

@codecov-commenter
Copy link

codecov-commenter commented Jan 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.16%. Comparing base (cf97b9f) to head (83c08fc).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #95      +/-   ##
==========================================
+ Coverage   96.07%   96.16%   +0.08%     
==========================================
  Files          10       10              
  Lines        1172     1172              
==========================================
+ Hits         1126     1127       +1     
+ Misses         46       45       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ericphanson
Copy link
Member Author

I think I'll do explicit-imports-jl here. I like that name and it matches the package (ExplicitImports.jl). If there's a standard convention for julia-tooling packages in a year or two I can do a breaking release and change it then, but I think in the short term there won't be.

@ericphanson ericphanson changed the title add app explicit-imports add app explicit-imports-jl Feb 1, 2025
@ericphanson ericphanson merged commit f07a495 into main Feb 1, 2025
7 checks passed
@ericphanson ericphanson deleted the eph/app branch February 1, 2025 18:29
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