Skip to content
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

Add (checked_)norem_div methods for integer types #116632

Closed
wants to merge 1 commit into from

Conversation

newpavlov
Copy link
Contributor

@newpavlov newpavlov commented Oct 11, 2023

The methods will be useful in cases when we need to ensure that numbers can be divided without remainder.

If the changes are good for inclusion, I will create a tracking issue for the feature.

ACP: rust-lang/libs-team#337

@rustbot
Copy link
Collaborator

rustbot commented Oct 11, 2023

r? @cuviper

(rustbot has picked a reviewer for you, use r? to override)

@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 Oct 11, 2023
@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@cuviper
Copy link
Member

cuviper commented Oct 11, 2023

@rustbot label +T-libs-api -T-libs
r? libs-api

@rustbot rustbot added T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. and removed T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Oct 11, 2023
@rustbot rustbot assigned joshtriplett and unassigned cuviper Oct 11, 2023
@newpavlov
Copy link
Contributor Author

@joshtriplett ping

@joshtriplett
Copy link
Member

r? libs

@rustbot rustbot added the T-libs Relevant to the library team, which will review and decide on the PR/issue. label Feb 11, 2024
@Mark-Simulacrum
Copy link
Member

This needs an ACP prior to landing for the new unstable API surface area.

@Mark-Simulacrum Mark-Simulacrum added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 17, 2024
@newpavlov newpavlov changed the title Add checked_norem_div method for integer types Add (checked_)norem_div methods for integer types Feb 17, 2024
@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@newpavlov
Copy link
Contributor Author

The ACP was created back in February, so I think the S-waiting-on-author tag can be removed?

@Dylan-DPC Dylan-DPC added S-waiting-on-ACP Status: PR has an ACP and is waiting for the ACP to complete. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Apr 10, 2024
@bors
Copy link
Contributor

bors commented Feb 4, 2025

☔ The latest upstream changes (presumably #135760) made this pull request unmergeable. Please resolve the merge conflicts.

@bors bors added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Feb 4, 2025
@newpavlov newpavlov closed this Feb 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-ACP Status: PR has an ACP and is waiting for the ACP to complete. S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-libs Relevant to the library team, which will review and decide on the PR/issue. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants