Skip to content

Add pub(restricted) example #856

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Jun 8, 2017
Merged

Conversation

cramertj
Copy link
Member

@cramertj
Copy link
Member Author

(Travis is obviously going to fail since the feature isn't stable yet.)

@steveklabnik
Copy link
Member

Yup! RBE deploys to master always, so we'll have to wait until the release to merge.

@cramertj
Copy link
Member Author

cramertj commented Mar 15, 2017

To add:

  • path in pub(in path) item must be some parent module of the item.
  • pub(super) and pub(self) are shortcuts for pub(in super) and pub(in self) respectively
  • pub(self) is the same thing as private

Edit: is it even worth mentioning pub(self)? why is that useful?

@steveklabnik
Copy link
Member

We'll be able to merge this Thursday; it now is green on beta 🎊

@steveklabnik steveklabnik merged commit 576efaf into rust-lang:master Jun 8, 2017
@steveklabnik
Copy link
Member

Woo!

@cramertj cramertj deleted the pub-restricted branch June 8, 2017 17:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants