Skip to content

Conversation

ada4a
Copy link
Contributor

@ada4a ada4a commented Aug 5, 2025

The issue that I saw made me rethink the lint design by quite a lot, so I decided to include that change in this PR

fixes #9000

changelog: [double_parens]: add structured suggestions
changelog: [double_parens]: fix FP when macros are involved

@rustbot
Copy link
Collaborator

rustbot commented Aug 5, 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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 5, 2025
@ada4a ada4a marked this pull request as draft August 22, 2025 15:13
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 22, 2025
@ada4a ada4a changed the title add structured suggestions to double_parens double_parens: add structured suggestions, fix bug Aug 22, 2025
@ada4a ada4a marked this pull request as ready for review August 22, 2025 16:31
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 22, 2025
@ada4a
Copy link
Contributor Author

ada4a commented Sep 7, 2025

@Jarcho friendly ping

@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 Sep 27, 2025
@rustbot
Copy link
Collaborator

rustbot commented Sep 28, 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 Sep 29, 2025

@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 Sep 29, 2025
Copy link
Contributor

@Jarcho Jarcho left a comment

Choose a reason for hiding this comment

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

@Jarcho Jarcho added this pull request to the merge queue Sep 29, 2025
Merged via the queue into rust-lang:master with commit 1a415e6 Sep 29, 2025
11 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 29, 2025
@ada4a ada4a deleted the double-parens branch September 29, 2025 23:44
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.

double_parens triggers on parenthesized call to macro that returns parenthesized expression
3 participants