Skip to content

Conversation

@RalfJung
Copy link
Member

There's both minimum and minimumNumber, so this seems worth clarifying.

Also use code font for these names to make it more clear that they are technical terms.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Nov 27, 2025
@rustbot
Copy link
Collaborator

rustbot commented Nov 27, 2025

r? @scottmcm

rustbot has assigned @scottmcm.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Copy link
Member

@hkBst hkBst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@scottmcm
Copy link
Member

Looks good, thanks!

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Nov 28, 2025

📌 Commit be22161 has been approved by scottmcm

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 Nov 28, 2025
bors added a commit that referenced this pull request Nov 29, 2025
Rollup of 5 pull requests

Successful merges:

 - #147362 (Avoid suggesting constrain the associated type with unknown type)
 - #149395 (float::minimum/maximum: say which exact IEEE operation this corresponds to)
 - #149396 (Remove outdated comment)
 - #149421 (Add a regression test for issue 129865)
 - #149424 (Update books)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 010da38 into rust-lang:main Nov 29, 2025
11 checks passed
@rustbot rustbot added this to the 1.93.0 milestone Nov 29, 2025
rust-timer added a commit that referenced this pull request Nov 29, 2025
Rollup merge of #149395 - RalfJung:float-minimum, r=scottmcm

float::minimum/maximum: say which exact IEEE operation this corresponds to

There's both `minimum` and `minimumNumber`, so this seems worth clarifying.

Also use code font for these names to make it more clear that they are technical terms.
@RalfJung RalfJung deleted the float-minimum branch November 29, 2025 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs Relevant 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.

5 participants