-
Notifications
You must be signed in to change notification settings - Fork 14.1k
Add doc aliases "vector" and "list" to Vec
#149232
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
I think list doesn't quite match that part of the policy because of what you said, but @bors r+ rollup |
Add doc aliases "vector" and "list" to `Vec` - "vector" is what "vec" is short form for. - "list" is a common name for this concept in a lot of other languages (`ArrayList` in Java, `list` in Python, `List` in C#). C++ uses the term for a linked list (`std::list`), so maybe this is contentious? I believe that this falls under [the doc alias policy](https://std-dev-guide.rust-lang.org/policy/doc-alias.html), and I think it's plausible that a newcomer would search for either of these terms, and expect to find `Vec`. r? libs
Rollup merge of #149232 - madsmtm:vec-doc-aliases, r=Noratrieb Add doc aliases "vector" and "list" to `Vec` - "vector" is what "vec" is short form for. - "list" is a common name for this concept in a lot of other languages (`ArrayList` in Java, `list` in Python, `List` in C#). C++ uses the term for a linked list (`std::list`), so maybe this is contentious? I believe that this falls under [the doc alias policy](https://std-dev-guide.rust-lang.org/policy/doc-alias.html), and I think it's plausible that a newcomer would search for either of these terms, and expect to find `Vec`. r? libs
Rollup of 3 pull requests Successful merges: - rust-lang/rust#149179 (Add regression test for 128705) - rust-lang/rust#149197 (validate usage of crate-level doc attributes) - rust-lang/rust#149232 (Add doc aliases "vector" and "list" to `Vec`) r? `@ghost` `@rustbot` modify labels: rollup
ArrayListin Java,listin Python,Listin C#). C++ uses the term for a linked list (std::list), so maybe this is contentious?I believe that this falls under the doc alias policy, and I think it's plausible that a newcomer would search for either of these terms, and expect to find
Vec.r? libs