Skip to content

Conversation

@alex-semenyuk
Copy link
Member

@alex-semenyuk alex-semenyuk commented Sep 12, 2024

As at #13355 we shoudn't trigger duplicated_attributes on duplicate reasons attr
changelog: [duplicated_attributes]: not trigger duplicated_attributes on duplicate reasons

@rustbot
Copy link
Collaborator

rustbot commented Sep 12, 2024

r? @Centri3

rustbot has assigned @Centri3.
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 Sep 12, 2024
Copy link
Contributor

@xFrednet xFrednet left a comment

Choose a reason for hiding this comment

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

r? xFrednet

@rustbot rustbot assigned xFrednet and unassigned Centri3 Sep 12, 2024
@xFrednet
Copy link
Contributor

Looks good to me, thank you for the fix :D


Roses are red,
Violets are blue,
reason = "this text"
allow two attributes now

@bors
Copy link
Contributor

bors commented Sep 12, 2024

📌 Commit 7097830 has been approved by xFrednet

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Sep 12, 2024

⌛ Testing commit 7097830 with merge e039e72...

@bors
Copy link
Contributor

bors commented Sep 12, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: xFrednet
Pushing e039e72 to master...

@bors bors merged commit e039e72 into rust-lang:master Sep 12, 2024
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.

5 participants