-
Notifications
You must be signed in to change notification settings - Fork 13.7k
Rollup of 18 pull requests #145566
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
Rollup of 18 pull requests #145566
Conversation
Comparison of paths and strings is expected to be possible and needed e.g. in tests. This change adds the impls os `PartialEq` between strings and paths, both owned and unsized, in both directions. ACP: rust-lang/libs-team#151
Migrate the standard library from using the external `cfg_if` crate to using the now-built-in `cfg_select` macro. This does not yet eliminate the dependency from `library/std/Cargo.toml`, because while the standard library itself no longer uses `cfg_if`, it also incorporates the `backtrace` crate, which does. Migration assisted by the following vim command (after selecting the full `cfg_if!` invocation): ``` '<,'>s/\(cfg_if::\)\?cfg_if/cfg_select/ | '<,'>s/^\( *\)} else {/\1}\r\1_ => {/c | '<,'>s/^\( *\)} else if #\[cfg(\(.*\))\] /\1}\r\1\2 => /e | '<,'>s/if #\[cfg(\(.*\))\] {/\1 => {/e ``` This is imperfect, but substantially accelerated the process. This prompts for confirmation on the `} else {` since that can also appear inside one of the arms. This also requires manual intervention to handle any multi-line conditions.
…` file if there is more than one
@bors try jobs=dist-loongarch64-musl |
This comment has been minimized.
This comment has been minimized.
Rollup of 18 pull requests try-job: dist-loongarch64-musl
🔒 Merge conflict This pull request and the master branch diverged in a way that cannot be automatically merged. Please rebase on top of the latest master branch, and let the reviewer approve again. How do I rebase?Assuming
You may also read Git Rebasing to Resolve Conflicts by Drew Blessing for a short tutorial. Please avoid the "Resolve conflicts" button on GitHub. It uses Sometimes step 4 will complete without asking for resolution. This is usually due to difference between how Error message
|
closing due to conflict |
Successful merges:
impl PartialEq<{str,String}> for {Path,PathBuf}
#140956 (impl PartialEq<{str,String}> for {Path,PathBuf}
)ip_from
#141744 (Stabilizeip_from
)as
casts as unreachable #144804 (Don't warn on never to anyas
casts as unreachable)tests/ui/issues/
tests to other subdirectories undertests/ui/
#144983 (Rehome 37tests/ui/issues/
tests to other subdirectories undertests/ui/
)rustdoc-js
tester would not pick the rightsearch.js
file if there is more than one #145359 (Fix bug whererustdoc-js
tester would not pick the rightsearch.js
file if there is more than one)unicode_data.rs
mention message #145486 (Fixunicode_data.rs
mention message)cfg_if
tocfg_select
#145489 (library: Migrate fromcfg_if
tocfg_select
)should_render
inPrintAttribute
derive #145493 (removeshould_render
inPrintAttribute
derive)lto
options from most to least optimizing #145533 (Reorderlto
options from most to least optimizing)()
inderive(From)
output. #145550 (Avoid using()
inderive(From)
output.)r? @ghost
@rustbot modify labels: rollup
Create a similar rollup