-
Notifications
You must be signed in to change notification settings - Fork 13.9k
Rollup of 10 pull requests #148356
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 10 pull requests #148356
Conversation
…renced by name, but not called.
…r=GuillaumeGomez Return spans out of `is_doc_comment` to reduce reliance on `.span()` on attributes r? `@GuillaumeGomez`
Remove boxes from ast list elements Less indirection should be better perf.
When matched against `Result` with copyable `Ok` variant, uncopyable scrutinee won't actually be moved. But `manual_unwrap_or`/`manual_unwrap_or_default` will force it to move, which can cause problem if scrutinee is used later. changelog: [`manual_unwrap_or`]: don't lint if not safe to move scrutinee changelog: [`manual_unwrap_or_default`]: don't lint if not safe to move scrutinee Signed-off-by: Zihan <[email protected]>
Update to direct link to the Code of Conduct changelog: none
The errors were unrelated to the lint...
- make them `verbose` -- the ones spanning multiple lines were especially egregious - give a more descriptive help message
…estions" This reverts commit 25881bf.
|
@bors r+ rollup=never p=5 |
Rollup of 10 pull requests Successful merges: - #135602 (Tweak output of missing lifetime on associated type) - #139751 (Implement pin-project in pattern matching for `&pin mut|const T`) - #142682 (Update bundled musl to 1.2.5) - #148171 (Simplify code to generate line numbers in highlight) - #148263 (Unpin `libc` and `rustix` in `compiler` and `rustbook`) - #148301 ([rustdoc search] Include extern crates when filtering on `import`) - #148330 (Don't require dlltool with the dummy backend on MinGW) - #148338 (cleanup: upstream dropped amx-transpose functionality) - #148340 (Clippy subtree update) - #148343 (`nonpoison::Condvar` should take `MutexGuard` by reference) r? `@ghost` `@rustbot` modify labels: rollup
|
💥 Test timed out |
|
@bors retry musl timeout |
|
☀️ Test successful - checks-actions |
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing d85276b (parent) -> fca2e94 (this PR) Test differencesShow 222 test diffsStage 0
Stage 1
Stage 2
Additionally, 186 doctest diffs were found. These are ignored, as they are noisy. Job group index
Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard fca2e941f86f51cd3c3254518f45e2145563f6be --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
📌 Perf builds for each rolled up PR:
previous master: d85276b256 In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
|
Finished benchmarking commit (fca2e94): comparison URL. Overall result: ✅ improvements - no action needed@rustbot label: -perf-regression Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (primary 2.3%, secondary -1.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary -3.4%, secondary -0.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary -1.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 476.797s -> 474.465s (-0.49%) |
Successful merges:
&pin mut|const T#139751 (Implement pin-project in pattern matching for&pin mut|const T)libcandrustixincompilerandrustbook#148263 (Unpinlibcandrustixincompilerandrustbook)import#148301 ([rustdoc search] Include extern crates when filtering onimport)nonpoison::Condvarshould takeMutexGuardby reference #148343 (nonpoison::Condvarshould takeMutexGuardby reference)r? @ghost
@rustbot modify labels: rollup
Create a similar rollup