-
Notifications
You must be signed in to change notification settings - Fork 2.7k
fix: doc-test failures #12055
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
fix: doc-test failures #12055
Conversation
|
r? @epage (rustbot has picked a reviewer for you, use r? to override) |
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.
Thanks for this.
Could you also fix failing doctests for other crates in this PR? I can see at least cargo-test-support is failing as well.
2cda053 to
30f4b17
Compare
30f4b17 to
ea0d487
Compare
|
Thank you very much for your guidance. I modified the documentation test case in Other crates did not find errors. |
Registry::new_handle doc-test failedThere 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.
Thanks! Only some bits of tiny issues left.
df7b432 to
ff7ff7d
Compare
|
Thanks! @bors r+ |
fix: doc-test failures ### What does this PR try to resolve? When I read the source code, I found that the `Registry::new_handle` document test failed and seemed to be missing parameters; There is no change in this [PR](https://github.com/rust-lang/cargo/pull/10592/files#diff-12973056cf99e8de997011a8738a6740a7dfd06142a73f122f3b35218db37351).
|
💥 Test timed out |
|
@bors retry macos stable builder is running extremely slow. |
|
☀️ Test successful - checks-actions |
Update cargo 10 commits in ac84010322a31f4a581dafe26258aa4ac8dea9cd..569b648b5831ae8a515e90c80843a5287c3304ef 2023-05-02 13:41:16 +0000 to 2023-05-05 15:49:44 +0000 - xtask-unpublished: output a markdown table (rust-lang/cargo#12085) - fix: hack around `libsysroot` instead of `libtest` (rust-lang/cargo#12088) - Optimize usage under rustup. (rust-lang/cargo#11917) - Update lock to normalize `home` dep (rust-lang/cargo#12084) - fix: doc-test failures (rust-lang/cargo#12055) - feat(cargo-metadata): add `workspace_default_members` (rust-lang/cargo#11978) - doc: clarify implications of `cargo-yank` (rust-lang/cargo#11862) - chore: Use `[workspace.dependencies]` (rust-lang/cargo#12057) - support for shallow clones and fetches with `gitoxide` (rust-lang/cargo#11840) - Build by PackageIdSpec, not name, to avoid ambiguity (rust-lang/cargo#12015) r? `@ghost`
What does this PR try to resolve?
When I read the source code, I found that the
Registry::new_handledocument test failed and seemed to be missing parameters;There is no change in this PR.