-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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 Url::into_string deprecation warning #9475
Conversation
(rust-highfive has picked a reviewer for you, use r? to override) |
r? @ehuss Would you mind also updating the version in |
Correct. It did in fact cause an issue when building on my side prior to running |
`url::Url::into_string` is deprecated. The `Into<String>` implementation is now used. Signed-off-by: Paul Mabileau <paulmabileau@hotmail.fr>
The base package, `cargo-test-support` and `mdman`. Signed-off-by: Paul Mabileau <paulmabileau@hotmail.fr>
I noticed it was also used in |
Thanks! @bors r+ |
📌 Commit a5b4b47 has been approved by |
☀️ Test successful - checks-actions |
Update cargo 8 commits in e51522ab3db23b0d8f1de54eb1f0113924896331..070e459c2d8b79c5b2ac5218064e7603329c92ae 2021-05-07 21:29:52 +0000 to 2021-05-11 18:12:23 +0000 - Fix rustdoc warnings (rust-lang/cargo#9468) - Improve performance of git status check in `cargo package`. (rust-lang/cargo#9478) - Link to the new rustc tests chapter. (rust-lang/cargo#9477) - Bump index cache version to deal with semver metadata version mismatch. (rust-lang/cargo#9476) - Fix Url::into_string deprecation warning (rust-lang/cargo#9475) - Fix rust-lang/cargo#4482 and rust-lang/cargo#9449: set Fossil ignore and clean settings locally (rust-lang/cargo#9469) - Improve two error messages (rust-lang/cargo#9472) - Fix `cargo install` with a semver metadata version. (rust-lang/cargo#9467)
is being emitted when running
cargo build
directly from thecrates/cargo-test-support
or thecrates/mdman
crate. This simply proposes a switch to the recommended method in order to resolve the deprecation warning.