-
Notifications
You must be signed in to change notification settings - Fork 479
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix Utilities.submodules for module aliases #852
Conversation
test/utilities.jl
Outdated
@@ -35,6 +35,17 @@ export OuterModule | |||
end | |||
end | |||
|
|||
module ModuleWithAlias | |||
using LinearAlgebra |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Create another test module instead? I believe this will pull in LinearAlgebra
and Random
as dependencies, which is not optimal.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. I do think there should be a module that's its own parent as a test case, so I'm using Main
for that now.
a920da1
to
d9cd1b9
Compare
d9cd1b9
to
59302a3
Compare
LGTM, thanks! |
Fix #851