Skip to content

clippy_dev: Parsing revamp part 4/N - #15978

Merged
llogiq merged 2 commits into
rust-lang:masterfrom
Jarcho:dev_parse4
Feb 22, 2026
Merged

clippy_dev: Parsing revamp part 4/N#15978
llogiq merged 2 commits into
rust-lang:masterfrom
Jarcho:dev_parse4

Conversation

@Jarcho

@Jarcho Jarcho commented Oct 29, 2025

Copy link
Copy Markdown
Contributor

Based on #15947

Simple change in this one. Lints are parsed into a map indexed by name in order to detect overlapping lint names. The panic message when a duplicate name is parsed is unhelpful right now. That will be fixed in a future PR along with improving all the other parsing errors.

changelog: none

@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 Oct 29, 2025
@rustbot

rustbot commented Oct 29, 2025

Copy link
Copy Markdown
Collaborator

r? @llogiq

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

@rustbot

This comment has been minimized.

@Jarcho Jarcho added the S-blocked Status: marked as blocked ❌ on something else such as an RFC or other implementation work label Oct 29, 2025
@rustbot

This comment has been minimized.

@Jarcho Jarcho removed the S-blocked Status: marked as blocked ❌ on something else such as an RFC or other implementation work label Feb 3, 2026
@rustbot

rustbot commented Feb 13, 2026

Copy link
Copy Markdown
Collaborator

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.

@Jarcho Jarcho removed the needs-fcp PRs that add, remove, or rename lints and need an FCP label Feb 13, 2026
@Jarcho

Jarcho commented Feb 13, 2026

Copy link
Copy Markdown
Contributor Author

Ping @llogiq.

The fpc label can be ignored. The lint list was just reordered (actually alphabetical for what little that's worth).

@llogiq

llogiq commented Feb 22, 2026

Copy link
Copy Markdown
Contributor

Thanks, merging.

@llogiq
llogiq added this pull request to the merge queue Feb 22, 2026
Merged via the queue into rust-lang:master with commit a57ca61 Feb 22, 2026
11 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Feb 22, 2026
github-merge-queue Bot pushed a commit that referenced this pull request Mar 5, 2026
…rt 5/N) (#15979)

Based on #15978

This will prevent some pointless merge conflict when multiple people add
lints to the same lint pass (e.g. methods).

changelog: None
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants