-
Notifications
You must be signed in to change notification settings - Fork 13.5k
DOCS: update reference about paths #30224
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
Conversation
wait, the rules are different: you can have any number of |
Should be correct now, but it's better to double check: the only thing worse than a missing reference is an incorrect reference :) |
|
||
* Paths starting with the keyword `super` begin resolution relative to the | ||
parent module. Each further identifier must resolve to an item. | ||
parent module. Each further identifier must resolve to an item. |
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.
This should have one space after the period.
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.
Yes, thanks. Fixed.
Hm, |
3badbea
to
c8b7e24
Compare
I hope this is correct and complete now ;) |
Can you extend example by inserting |
Are you sure this is necessary? There are such examples just before this point. My idea is that in previous bullets we discuss simple common cases, and that then we give a separate more complicated example with several |
Sorry, I didn't read the context. Ignore my comment. |
@bors: r+ rollup |
📌 Commit c8b7e24 has been approved by |
Make clear that `super` may be included in the path several times. r? @steveklabnik
Make clear that
super
may be included in the path several times.r? @steveklabnik