Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Rollup of 8 pull requests #113256

Merged
merged 16 commits into from
Jul 2, 2023
Merged

Rollup of 8 pull requests #113256

merged 16 commits into from
Jul 2, 2023

Conversation

matthiaskrgr
Copy link
Member

Successful merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

lizhanhui and others added 16 commits June 29, 2023 04:21
Signed-off-by: Li Zhanhui <lizhanhui@gmail.com>
The args for unittest and cargo test are mutually incompatible. Suggest that people use `python -m unittest ...` manually instead.

This also changes `bootstrap_test.py` to be easier to run standalone; see the commit for details.
…oc_example, r=Mark-Simulacrum

Fix document examples of Vec::from_raw_parts and Vec::from_raw_parts_in

These two examples are misplaced.
…lor-18, r=notriddle

Migrate GUI colors test to original CSS color format

Follow-up of rust-lang#111459.

r? ``@notriddle``
…, r=Nilstrieb

Fix `dropping_copy_types` lint from linting in match-arm with side-effects

This PR fixes an issue with the `dropping_copy_types` and `dropping_references` lints when not all patterns that can have side-effects were detected and ignored.

Nearly _fixes_ rust-lang#112653 (will need beta-backport to completely fix the issue)
r? ``@Nilstrieb``
…clubby789

Don't pass --test-args to `python -m unitest`

The args for unittest and cargo test are mutually incompatible. Suggest that people use `python -m unittest ...` manually instead.

This also changes `bootstrap_test.py` to be easier to run standalone; see the commit for details.

r? `@clubby789` cc rust-lang#112281 (comment)
…ts, r=jyn514

Make Rust Analyzer tests faster by compiling less code

This should bring down the test duration of RA significantly.
Document tracking issue for rustdoc `show-type-layout`

Tracking issue: rust-lang#113248

``@rustbot`` label +T-rustdoc +A-docs
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. rollup A PR which is a rollup labels Jul 2, 2023
@matthiaskrgr
Copy link
Member Author

@bors r+ rollup=never p=8

@bors
Copy link
Contributor

bors commented Jul 2, 2023

📌 Commit fde4102 has been approved by matthiaskrgr

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 2, 2023
@bors
Copy link
Contributor

bors commented Jul 2, 2023

⌛ Testing commit fde4102 with merge 4c3f8c7...

@bors
Copy link
Contributor

bors commented Jul 2, 2023

☀️ Test successful - checks-actions
Approved by: matthiaskrgr
Pushing 4c3f8c7 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Jul 2, 2023
@bors bors merged commit 4c3f8c7 into rust-lang:master Jul 2, 2023
@rustbot rustbot added this to the 1.72.0 milestone Jul 2, 2023
@rust-timer
Copy link
Collaborator

📌 Perf builds for each rolled up PR:

PR# Message Perf Build Sha
#113147 Fix document examples of Vec::from_raw_parts and Vec::from_… a87f08f92e76fe86061c38100a31f868a2ebc358 (link)
#113202 std docs: factorize literal in Barrier example a1acc6f6c3c6c5165c1207c1752ffc26ec16b3a7 (link)
#113228 Migrate GUI colors test to original CSS color format 980d6e95379a4cac265023974ab8e6170a4daf9f (link)
#113231 Fix dropping_copy_types lint from linting in match-arm wi… 2cdf280d6e718d0b550857d3836dfe343f5fd613 (link)
#113233 Update chalk c2c019a404c34e28175a40a3b8e36a1b425aa928 (link)
#113234 Don't pass --test-args to python -m unitest 79f6b078fb67139a7dfca9da7f448e1dcaddc6b3 (link)
#113244 Make Rust Analyzer tests faster by compiling less code ad119e85ab85823846709a94e6e92ff72cb92551 (link)
#113249 Document tracking issue for rustdoc show-type-layout 11eb42f3d3b6948b1e5c880cad7661975304dad7 (link)

previous master: 72b2101434

In the case of a perf regression, run the following command for each PR you suspect might be the cause: @rust-timer build $SHA

@rust-timer
Copy link
Collaborator

Finished benchmarking commit (4c3f8c7): comparison URL.

Overall result: ❌ regressions - no action needed

@rustbot label: -perf-regression

Instruction count

This is a highly reliable metric that was used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)
0.4% [0.4%, 0.4%] 1
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) 0.4% [0.4%, 0.4%] 1

Max RSS (memory usage)

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)
1.1% [0.8%, 1.3%] 2
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
-4.1% [-4.1%, -4.1%] 1
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) -0.7% [-4.1%, 1.3%] 3

Cycles

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
3.2% [2.6%, 4.2%] 4
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) - - 0

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 662.283s -> 662.06s (-0.03%)

@matthiaskrgr matthiaskrgr deleted the rollup-poex8w2 branch March 16, 2024 18:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
merged-by-bors This PR was explicitly merged by bors. rollup A PR which is a rollup S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.