-
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
Update mdbook version #39966
Update mdbook version #39966
Conversation
r? @brson (rust_highfive has picked a reviewer for you, use r? to override) |
@bors: r+ rollup |
📌 Commit 4cc4944 has been approved by |
This version of mdbook includes https://github.com/azerupi/mdBook/pull/207 , which is needed so that we can start doing linkchecker on the various books.
In order to update mdbook, we have to update other stuff too.
4cc4944
to
2795de3
Compare
@bors: r=GuillaumeGomez rollup I didn't notice we needed to update other deps too, oops 😓 |
📌 Commit 2795de3 has been approved by |
@bors: p=1 (All the other commits in the queue are r+'d by me, I'd like this one to land first) |
⌛ Testing commit 2795de3 with merge bcd62ac... |
💔 Test failed - status-appveyor |
@bors: retry |
⌛ Testing commit 2795de3 with merge 322f0d7... |
Update mdbook version This version of mdbook includes https://github.com/azerupi/mdBook/pull/207 , which is needed so that we can start doing linkchecker on the various books.
☀️ Test successful - status-appveyor, status-travis |
This reverts commit 7f1d1c6. The original commit was created because mdBook and rustdoc had different generation algorithms for header links; now with rust-lang#39966 , the algorithms are the same. So let's undo this change. ... when I came across this problem, I said "eh, this isn't fun, but it doesn't take that long." I probably should have just actually taken the time to fix upstream, given that they were amenable. Oh well!
Previously, mdBook used JavaScript to add header links, so we skipped checking the book. As of rust-lang#39966, it no longer does, so we can start checking again. There is a twist, though: it uses name instead of id, so let's test for both. They're both valid links anyway, so it's good to have the checker check anyway.
This reverts commit 7f1d1c6d9a7be5e427bace30e740b16b25f25c92. The original commit was created because mdBook and rustdoc had different generation algorithms for header links; now with rust-lang/rust#39966 , the algorithms are the same. So let's undo this change. ... when I came across this problem, I said "eh, this isn't fun, but it doesn't take that long." I probably should have just actually taken the time to fix upstream, given that they were amenable. Oh well!
This version of mdbook includes
https://github.com/azerupi/mdBook/pull/207 , which is needed so that
we can start doing linkchecker on the various books.