Skip to content

Conversation

@ada4a
Copy link
Contributor

@ada4a ada4a commented Oct 22, 2025

As described in #15528 (comment)

changelog: none

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

rustbot commented Oct 22, 2025

r? @Jarcho

rustbot has assigned @Jarcho.
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 ada4a changed the title Repeated is diagnostic item New internal lint: repeated_is_diagnostic_item Oct 22, 2025
@@ -0,0 +1,218 @@
use crate::internal_paths::MAYBE_DEF;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The lint, thankfully, gives a bunch warnings on dogfood, but for some reason, cargo dev dogfood --fix doesn't seem to autofix them; I've checked that Applicability remains MachineApplicable even after all the suggestions are created.

Any ideas?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I went ahead and applied the fixes by hand... Still, would like to avoid the need to do this in the future

@ada4a ada4a force-pushed the repeated_is_diagnostic_item branch from c5c5bba to 706d519 Compare November 8, 2025 11:11
@rustbot
Copy link
Collaborator

rustbot commented Nov 8, 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 ada4a marked this pull request as draft November 9, 2025 19:26
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Nov 9, 2025
@ada4a ada4a force-pushed the repeated_is_diagnostic_item branch from 706d519 to 0560075 Compare November 9, 2025 19:52
@ada4a ada4a marked this pull request as ready for review November 9, 2025 19:58
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Nov 9, 2025
@ada4a ada4a force-pushed the repeated_is_diagnostic_item branch from 0560075 to eae5601 Compare November 9, 2025 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants