-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
doc: add release notes to standalone doc bundle #117888
Conversation
r? @onur-ozkan (rustbot has picked a reviewer for you, use r? to override) |
This comment has been minimized.
This comment has been minimized.
7df8467
to
70ba2a2
Compare
This comment has been minimized.
This comment has been minimized.
This is a workaround for rust-lang#101714 on top of being a useful addition in its own right. It is intended to change the "canonical URL" for viewing the release notes from GitHub, which is relatively slow, to a pre-rendered HTML file that loads from the same CDN as the standard library docs. It also means you get a copy of the release notes when installing the rust-docs with rustup.
70ba2a2
to
29f5d2c
Compare
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.
Nice workaround, LGTM!
r=me with the nit suggestion
@bors rollup=iffy |
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Onur Özkan <[email protected]>
c59660c
to
a3d9003
Compare
While I like it very much, I don't want to make the decision alone, so I would like one more person to review it. r? Mark-Simulacrum |
r=me beyond the small comment |
@bors r=Mark-Simulacrum |
☀️ Test successful - checks-actions |
Finished benchmarking commit (290fc68): comparison URL. Overall result: ❌ regressions - no action needed@rustbot label: -perf-regression Instruction countThis is a highly reliable metric that was used to determine the overall result at the top of this comment.
Max RSS (memory usage)This benchmark run did not return any relevant results for this metric. CyclesThis benchmark run did not return any relevant results for this metric. Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 676.664s -> 677.922s (0.19%) |
Now that https://dev-doc.rust-lang.org/1.76.0/releases.html exists, we can use it and not have to worry about rust-lang/rust#101714 any more. This is a follow-up for rust-lang/rust#117888
Now that https://dev-doc.rust-lang.org/1.76.0/releases.html exists, we can use it and not have to worry about rust-lang/rust#101714 any more. This is a follow-up for rust-lang/rust#117888
Preview: http://notriddle.com/rustdoc-html-demo-5/release-notes/releases.html
This is a workaround for #101714 on top of being a useful addition in its own right. It is intended to change the "canonical URL" for viewing the release notes from GitHub, which is relatively slow, to a pre-rendered HTML file that loads from the same CDN as the standard library docs. It also means you get a copy of the release notes when installing the rust-docs with rustup.