-
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
Add missing urls and small nits #39513
Conversation
r? @sfackler (rust_highfive has picked a reviewer for you, use r? to override) |
r? @frewsxcv |
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.
@bors r+ rollup
@bors r+ rollup |
📌 Commit 5fb89bc has been approved by |
@bors r-
|
5c57475
to
226be6b
Compare
226be6b
to
2281290
Compare
Updated. |
Strange, it works when tested with the generated docs... |
It's probably because the docs are generated in two places. The |
Makes sense. So I guess we're stuck in here. I'll try to solve this issue later on. |
☔ The latest upstream changes (presumably #40748) made this pull request unmergeable. Please resolve the merge conflicts. |
Let's give this one a close, since it's old and has problems. Thanks @GuillaumeGomez ! |
r? @frewsxcv