Skip to content

Rename unecessary_nonzero_get to needless_nonzero_get - #17538

Merged
rustbot merged 1 commit into
rust-lang:masterfrom
Gri-ffin:rename_unecessary_nonzero_get
Aug 11, 2026
Merged

Rename unecessary_nonzero_get to needless_nonzero_get#17538
rustbot merged 1 commit into
rust-lang:masterfrom
Gri-ffin:rename_unecessary_nonzero_get

Conversation

@Gri-ffin

@Gri-ffin Gri-ffin commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Ref: #t-clippy/fcp > FCP rust-clippy#17499: New lint/nonzero operators and metho…

r? @samueltardieu

changelog: [unnecessary_nonzero_get]: rename to needless_nonzero_get (same release)

@rustbot rustbot added needs-fcp PRs that add, remove, or rename lints and need an FCP S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Aug 10, 2026
@rustbot

rustbot commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

samueltardieu is not on the review rotation at the moment.
They may take a while to respond.

@samueltardieu samueltardieu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Since it has not been released even as beta yet, the rename should act as if the lint had the new name in the first place, not as a lint renaming.

View changes since this review

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Aug 11, 2026
@rustbot

rustbot commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Reminder, once the PR becomes ready for a review, use @rustbot ready.

@Gri-ffin
Gri-ffin force-pushed the rename_unecessary_nonzero_get branch from 51110b7 to 7c10243 Compare August 11, 2026 19:29
@Gri-ffin

Copy link
Copy Markdown
Contributor Author

@rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties and removed S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) labels Aug 11, 2026
@Gri-ffin

Gri-ffin commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

should I then drop the changelog in the pr description?

@samueltardieu

Copy link
Copy Markdown
Member

should I then drop the changelog in the pr description?

Just add "same release” to it.

@samueltardieu samueltardieu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You can merge it after the changelog has been updated.

@rustbot delegate

View changes since this review

@rustbot

rustbot commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

✌️ @Gri-ffin, you can now merge this pull request!

If @samueltardieu told you to merge after making some further change, then please make that change and post @rustbot merge.

View changes since this delegation

@Gri-ffin

Copy link
Copy Markdown
Contributor Author

@rustbot merge

@rustbot
rustbot added this pull request to the merge queue Aug 11, 2026
Merged via the queue into rust-lang:master with commit 907246a Aug 11, 2026
13 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 11, 2026
@Gri-ffin
Gri-ffin deleted the rename_unecessary_nonzero_get branch August 11, 2026 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-fcp PRs that add, remove, or rename lints and need an FCP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants