Skip to content

Rollup of 3 pull requests - #146066

Closed
jhpratt wants to merge 8 commits into
rust-lang:masterfrom
jhpratt:rollup-vh43plw
Closed

Rollup of 3 pull requests#146066
jhpratt wants to merge 8 commits into
rust-lang:masterfrom
jhpratt:rollup-vh43plw

Conversation

@jhpratt

@jhprattjhpratt commented Aug 31, 2025

Copy link
Copy Markdown
Member

Successful merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

connortsui20and others added 8 commits August 30, 2025 10:48
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
Co-authored-by: Jonas Böttiger <jonasboettiger@icloud.com>
Add `Bound::copied`
Tracking Issue: rust-lang#145966
Some questions:
- [x] Should I update the documentation for `cloned` to actual used a `Clone` type instead of an integer?
- [x] I removed the `must_use` since this is a cheap copy, does that make sense?
…s-for-ret-ty-error, r=estebank
Suggest method name with maybe ty mismatch
Fixesrust-lang#146008
I think it's ok to suggest method name even when the return ty mismatch, since we only reporting in `Applicability::MaybeIncorrect`, user may add `()` and continue to fix following errors.
r? `@estebank`
…r=RalfJung
Change std f32 test to pass under Miri
Adds `APPROX_DELTA` to 2 tests of `f32::log` reported in [#miri > Miri test-libstd Failure (2025-08)](https://rust-lang.zulipchat.com/#narrow/channel/269128-miri/topic/Miri.20test-libstd.20Failure.20.282025-08.29).
I changed 2 lines, the first one is the actual failure. The second one is precautionary.
@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. 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. rollup A PR which is a rollup labels Aug 31, 2025
@jhpratt

Copy link
Copy Markdown
MemberAuthor

@bors r+ rollup=never p=3

@bors

bors commented Aug 31, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit 7372132 has been approved by jhpratt

It is now in the queue for this repository.

@borsbors 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 Aug 31, 2025
@bors

bors commented Aug 31, 2025

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 7372132 with merge 01d8fd6...

bors added a commit that referenced this pull request Aug 31, 2025
Rollup of 3 pull requests
Successful merges:
- #145968 (Add `Bound::copied`)
- #146046 (Suggest method name with maybe ty mismatch)
- #146051 (Change std f32 test to pass under Miri)
r? `@ghost`
`@rustbot` modify labels: rollup
@rust-log-analyzer

Copy link
Copy Markdown
Collaborator

The job x86_64-gnu-aux failed! Check out the build log: (web)(plain enhanced)(plain)

Click to see the possible cause of the failure (guessed by this bot)

@bors

bors commented Sep 1, 2025

Copy link
Copy Markdown
Collaborator

💔 Test failed - checks-actions

@borsbors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Sep 1, 2025
@Zalathar

Copy link
Copy Markdown
Member

Pretty sure that's a flaky failure, but let's close this in favour of a bigger rollup now that there's more stuff in the queue.

@ZalatharZalathar closed this Sep 1, 2025
@rustbotrustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 1, 2025
@jhpratt
jhpratt deleted the rollup-vh43plw branch September 1, 2025 04:08
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rollupA PR which is a rollupT-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@jhpratt@bors@rust-log-analyzer@Zalathar@rustbot@connortsui20@chenyukang@LorrensP-2158466