-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Treat header as first paragraph for shortened markdown descriptions #82962
Conversation
"The Rust Standard LibraryThe Rust Standard Library is the …" is an awful description.
(rust-highfive has picked a reviewer for you, use r? to override) |
This comment has been minimized.
This comment has been minimized.
It's kinda silly using serde seq for fixed-length stuff.
97a9463
to
66b6504
Compare
let mut seq = serializer.serialize_seq(None)?; | ||
seq.serialize_element(&self.ty.name)?; | ||
let x: ItemType = self.kind.into(); | ||
seq.serialize_element(&x)?; | ||
seq.end() | ||
(&self.ty.name, ItemType::from(self.kind)).serialize(serializer) |
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.
What does this change do?
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.
It shouldn't actually cause any behaviour changes. It just removes lines of code.
@bors r+ |
📌 Commit 66b6504 has been approved by |
Rollup of 9 pull requests Successful merges: - rust-lang#81309 (always eagerly eval consts in Relate) - rust-lang#82217 (Edition-specific preludes) - rust-lang#82807 (rustdoc: Remove redundant enableSearchInput function) - rust-lang#82924 (WASI: Switch to crt1-command.o to enable support for new-style commands) - rust-lang#82949 (Do not attempt to unlock envlock in child process after a fork.) - rust-lang#82955 (fix: wrong word) - rust-lang#82962 (Treat header as first paragraph for shortened markdown descriptions) - rust-lang#82976 (fix error message for copy(_nonoverlapping) overflow) - rust-lang#82977 (Rename `Option::get_or_default` to `get_or_insert_default`) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
"The Rust Standard LibraryThe Rust Standard Library is the …" is an awful description.