Skip to content

Conversation

@ada4a
Copy link
Contributor

@ada4a ada4a commented Oct 19, 2025

According to the lint naming guidelines1, lint names should use the
plural form.

changelog: [empty_enum]: rename to empty_enums

Footnotes

  1. https://rust-lang.github.io/rfcs/0344-conventions-galore.html#lints

@rustbot rustbot added needs-fcp S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Oct 19, 2025
@rustbot
Copy link
Collaborator

rustbot commented Oct 19, 2025

r? @dswij

rustbot has assigned @dswij.
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

@ada4a
Copy link
Contributor Author

ada4a commented Oct 19, 2025

Depends on #15911, so @rustbot blocked

@rustbot rustbot added S-blocked Status: marked as blocked ❌ on something else such as an RFC or other implementation work and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Oct 19, 2025
@ada4a ada4a force-pushed the empty_enums branch 2 times, most recently from 52c60d6 to eb25f28 Compare October 19, 2025 22:48
@ada4a
Copy link
Contributor Author

ada4a commented Oct 20, 2025

PR was merged, so @rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties and removed S-blocked Status: marked as blocked ❌ on something else such as an RFC or other implementation work labels Oct 20, 2025
@dswij
Copy link
Member

dswij commented Oct 20, 2025

@ada4a thanks! Can we squash the commits?

According to the lint naming guidelines[^1], lint names should use the plural form. [^1]: https://rust-lang.github.io/rfcs/0344-conventions-galore.html#lints
@rustbot
Copy link
Collaborator

rustbot commented Oct 20, 2025

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@ada4a
Copy link
Contributor Author

ada4a commented Oct 20, 2025

It was just one commit, but on top of the two commits from the previous PR. Rebased^^

@ada4a

This comment was marked as outdated.

@dswij dswij added this pull request to the merge queue Oct 23, 2025
Merged via the queue into rust-lang:master with commit 09c3237 Oct 23, 2025
11 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Oct 23, 2025
@ada4a ada4a deleted the empty_enums branch October 23, 2025 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants