-
Notifications
You must be signed in to change notification settings - Fork 12.6k
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
Fixing rustDoc for LayoutError. #131512
Fixing rustDoc for LayoutError. #131512
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @jhpratt (or someone else) some time within the next two weeks. Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (
|
For consistency purposes I wanted to check that this isn't out of the ordinary, as the existing documentation is correct. Many other errors in the standard library provide similar context, so this LGTM. @bors r+ rollup |
Fixing rustDoc for LayoutError. I started reading the the std lib from start to finish and noticed that this rustdoc comment wasn't correct.
💔 Test failed - checks-actions |
@bors retry |
Fixing rustDoc for LayoutError. I started reading the the std lib from start to finish and noticed that this rustdoc comment wasn't correct.
Fixing rustDoc for LayoutError. I started reading the the std lib from start to finish and noticed that this rustdoc comment wasn't correct.
…iaskrgr Rollup of 5 pull requests Successful merges: - rust-lang#131464 (Update wasm-component-ld to 0.5.10) - rust-lang#131498 (Consider outermost const-anon in `non_local_def` lint) - rust-lang#131512 (Fixing rustDoc for LayoutError.) - rust-lang#131529 (rustdoc-json-types: fix typo in comment) - rust-lang#131531 (move dummy commit logic into x86_64-gnu-llvm-18) Failed merges: - rust-lang#131476 (coverage: Include the highest counter ID seen in `.cov-map` dumps) r? `@ghost` `@rustbot` modify labels: rollup
…iaskrgr Rollup of 6 pull requests Successful merges: - rust-lang#131464 (Update wasm-component-ld to 0.5.10) - rust-lang#131476 (coverage: Include the highest counter ID seen in `.cov-map` dumps) - rust-lang#131497 (Add myself to bootstrap review rotation) - rust-lang#131498 (Consider outermost const-anon in `non_local_def` lint) - rust-lang#131512 (Fixing rustDoc for LayoutError.) - rust-lang#131529 (rustdoc-json-types: fix typo in comment) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of rust-lang#131512 - j7nw4r:master, r=jhpratt Fixing rustDoc for LayoutError. I started reading the the std lib from start to finish and noticed that this rustdoc comment wasn't correct.
@bors r- still in the queue |
Is there something that I need to do? I see the tag says waiting on author. |
Nope! The PR has been merged. The label is a side effect of the bot acting up and trying to merge it twice. |
Ah ok. Thanks a lot. This is my first commit to the rust language. I guess I'm just a little eager. |
No worries! You can actually see the change if you view the docs on nightly. https://doc.rust-lang.org/nightly/std/alloc/struct.LayoutError.html |
I started reading the the std lib from start to finish and noticed that this rustdoc comment wasn't correct.