- 
                Notifications
    You must be signed in to change notification settings 
- Fork 13.9k
Rollup of 4 pull requests #148245
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 4 pull requests #148245
Conversation
Reorganize rustdoc tests into their correct subdirectories for better categorization: - Move lint-related tests to rustdoc-ui/lints/ - Move intra-doc link tests to rustdoc-ui/intra-doc/ - Move deref-related tests to rustdoc-ui/deref/ - Move doc-cfg test to rustdoc/doc-cfg/ This improves test organization and makes it easier to find tests related to specific rustdoc functionality.
Slightly reword the issue prioritization and beta backport Zulip messages
…dirs, r=GuillaumeGomez Move rustdoc tests to appropriate subdirectories ## Summary This PR reorganizes rustdoc tests into their correct subdirectories for better categorization and maintainability. ## Changes - Moved 6 lint-related tests to `tests/rustdoc-ui/lints/` - Moved 2 intra-doc link tests to `tests/rustdoc-ui/intra-doc/` - Moved 3 deref-related tests to `tests/rustdoc-ui/deref/` - Moved 1 doc-cfg test to `tests/rustdoc/doc-cfg/` ## Tests All moved tests have been verified and pass in their new locations.
…r=Urgau,jieyouxu Update T-compiler/ops Zulip messages Slightly reword the issue prioritization and beta backport Zulip messages r? `@jieyouxu` wdyt? (we discussed together this wording)
…, r=nnethercote Run regression test for rust-lang#147964 on next solver too Original regression: rust-lang#147964 Longer-term tracking of issue: rust-lang#148028 For now, this just makes sure that whatever tests we keep are working the same under both the current solver and the next solver, until we decide what to do with them.
citool: Always print a signed percentage in job duration changes I found myself confused by these percentage deltas, because it's not immediately obvious whether a change of 110% means “a little bit worse” (+10%, 110% overall) or ”more than twice as long” (+110%, 210% overall). The correct interpretation is “more than twice as long”. Including a sign for positive percentages should hopefully make it clearer that they are relative to a baseline of ±0%, not a baseline of 100%. Manually tested with: ```text cargo run --manifest-path src/ci/citool/Cargo.toml post-merge-report df984ed c9537a9 ``` r? Kobzol
| @bors r+ rollup=never p=5 | 
| ☀️ Test successful - checks-actions | 
| 📌 Perf builds for each rolled up PR: 
 previous master: c6d42d774d In the case of a perf regression, run the following command for each PR you suspect might be the cause:  | 
| 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 c6d42d7 (parent) -> 292db4a (this PR) Test differencesShow 56 test diffsStage 1
 Stage 2
 Additionally, 2 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 292db4a13ce3f329579a581b424c6cec3b1cbd2b --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 | 
| Finished benchmarking commit (292db4a): 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 (secondary -3.0%)A less reliable metric. May be of interest, but not used to determine the overall result above. 
 CyclesResults (primary -2.9%, secondary 0.3%)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: 475.179s -> 476.353s (0.25%) | 
Successful merges:
as_refexists for reference #147964 on next solver too)r? @ghost
@rustbot modify labels: rollup
Create a similar rollup