-
Notifications
You must be signed in to change notification settings - Fork 530
Add note about anon params #421
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
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.
@steveklabnik I'd like the reference to be more specification-like at some point including using BNF formalisms to define things like these; but for now, this seems good enough. :) |
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.
Notes are for non-normative information. Please change it to "Edition Differences" or something similar. And bold it. And then, if you're up to it, add it to conventions.md
. If not, I can add it in another PR.
I sent a PR against your branch. mark-i-m#1 |
Thanks @Havvy! Sorry I'm a bit sluggish. I'm in a bit of time crunch. |
No need to apologize. You're not moving slower than anybody else on this repo. I just felt like making the changes, so I did.. |
@Havvy Perhaps this can be merged now? |
I generally don't like merging things where I've sent commits. |
Thank you both! |
cc @Centril @steveklabnik rust-lang/rust#41686