Skip to content
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

Add a reexported keyword for use with using, import, and importall #5626

Closed
wants to merge 1 commit into from

Conversation

simonster
Copy link
Member

This is an alternative to #5608 that is similar to @vtjnash's proposal from #1986, although I didn't implement the as part of that proposal. Syntax is:

using MyModule reexported, ...
import MyModule.fn reexported, ...
importall MyModule reexported, ...

I chose this approach over the others mentioned in #1986 because it wasn't clear to me what should happen if you try to reexport MyModule or exportall MyModule without first having called importall MyModule or using MyModule. It also avoids an additional reserved keyword since it is only applicable after using, import, or importall. I am not convinced it is a better option than #5608 (it's not any easier to use and it's a lot more code), but hopefully one of these can make it into Base.

Syntax is:

using MyModule reexported, ...
import MyModule.fn reexported, ...
importall MyModule reexported, ...
@oscardssmith
Copy link
Member

Closing as 1.6 has a similar feature.

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