-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Don't silently ignore errors that happen during rendering #104995
Conversation
This caught a real, existing bug.
r? @CraftSpider (rustbot has picked a reviewer for you, use r? to override) |
Adding |
Oof, good catch @bors r+ (Unsure if this ought to be marked as rollup=always) |
@workingjubilee points out we should likely do the same thing for |
Don't silently ignore errors that happen during rendering This caught a real, existing bug. Found in rust-lang#104994
@@ -8,3 +8,4 @@ pub struct S { | |||
} |
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.
update test's file comment
Not sure I can still do this, but: |
You can and thanks for it! We'll try to take a look soon. |
…e-rustdoc-errors, r=notriddle Dont silently ignore rustdoc errors I applied the suggestions from rust-lang#104995 and also checked the rustdoc-ui error but couldn't reproduce it. r? `@notriddle`
I have added the comment you mentioned into #105427. The CI failed though on linux i686, not sure why. I ignored this platform in the test and just started a try build to see if this specific to this platform or not. Please feel free to close my PR if you intend to update yours. |
…e-rustdoc-errors, r=notriddle Dont silently ignore rustdoc errors I applied the suggestions from rust-lang#104995 and also checked the rustdoc-ui error but couldn't reproduce it. r? `@notriddle`
☔ The latest upstream changes (presumably #105486) made this pull request unmergeable. Please resolve the merge conflicts. |
…-errors, r=notriddle Dont silently ignore rustdoc errors I applied the suggestions from rust-lang/rust#104995 and also checked the rustdoc-ui error but couldn't reproduce it. r? `@notriddle`
This caught a real, existing bug.
Found in #104994