Conversation
|
(rust_highfive has picked a reviewer for you, use r? to override) |
|
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
Can you update this to the latest master? There are some fixes I'd like to get in for 1.39. |
774999c to
7419050
Compare
|
@ehuss Updated |
|
Thanks! @bors r+ |
|
📌 Commit 7419050 has been approved by |
Update Cargo This pulls in rust-lang/cargo#7159, which ensures that documenting proc macros works correctly.
|
Blocked on rust-lang/rls#1554 |
|
Oh, sorry, that's not quite the right approach to solve that problem. That PR won't have any effect, because rust-lang/rust does not use that version of cargo in rls. There is a patch override so it always uses the version here. I posted rust-lang/cargo#7385 with a fix. We just have to be careful to use the same set of features across dependencies. |
|
OK, 7385 has been merged, so you should be able to update the submodule, update diff --git a/src/tools/rustc-workspace-hack/Cargo.toml b/src/tools/rustc-workspace-hack/Cargo.toml
index 930279c0ca..bafddee99f 100644
--- a/src/tools/rustc-workspace-hack/Cargo.toml
+++ b/src/tools/rustc-workspace-hack/Cargo.toml
@@ -62,7 +62,7 @@ crossbeam-utils = { version = "0.6.5", features = ["nightly"] }
serde = { version = "1.0.82", features = ['derive'] }
serde_json = { version = "1.0.31", features = ["raw_value"] }
smallvec = { version = "0.6", features = ['union', 'may_dangle'] }
-
+url = { version = "2.0", features = ['serde'] }
[target.'cfg(not(windows))'.dependencies]
openssl = { version = "0.10.12", optional = true }That way, the tools should have the same features. |
This pulls in rust-lang/cargo#7159, which ensures that documenting proc macros works correctly.
7419050 to
d440e13
Compare
|
@ehuss Updated |
|
Thanks! |
|
📌 Commit d440e13 has been approved by |
Update Cargo This pulls in rust-lang/cargo#7159, which ensures that documenting proc macros works correctly.
Rollup of 5 pull requests Successful merges: - #63630 (Update installed compiler dependencies) - #64536 (Update Cargo) - #64554 (Polonius: more `ui` test suite fixes) - #64566 (A more generic interface for dataflow analysis) - #64591 (Fix a minor grammar nit, update UI tests) Failed merges: r? @ghost
Now that rust-lang/rust#64536 has landed, all of the bugs with proc macro documentation have now been fixed (currently only on the latest nightly). We can now use `#[doc(inline)]` on proc macro imports, which works just like any other import.
Now that rust-lang/rust#64536 has landed, all of the bugs with proc macro documentation have now been fixed (currently only on the latest nightly). We can now use `#[doc(inline)]` on proc macro imports, which works just like any other import.
Now that rust-lang/rust#64536 has landed, all of the bugs with proc macro documentation have now been fixed (currently only on the latest nightly). We can now use `#[doc(inline)]` on proc macro imports, which works just like any other import.
98: Properly inline documentation and use intra-crate lints r=taiki-e a=Aaron1011 Now that rust-lang/rust#64536 has landed, all of the bugs with proc macro documentation have now been fixed (currently only on the latest nightly). We can now use `#[doc(inline)]` on proc macro imports, which works just like any other import. Co-authored-by: Aaron Hill <aa1ronham@gmail.com>
This pulls in rust-lang/cargo#7159, which
ensures that documenting proc macros works correctly.