Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change applicability of assigning_clones to Unspecified #12554

Merged
merged 1 commit into from
Mar 25, 2024

Conversation

Kobzol
Copy link
Contributor

@Kobzol Kobzol commented Mar 25, 2024

Before we deal with #12473 and the borrow checker errors, I think that it would be better to downgrade this lint, since it can break code.

changelog: Change the applicability of assigning_clones to Unspecified

r? @blyxyas

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Mar 25, 2024
@blyxyas
Copy link
Member

blyxyas commented Mar 25, 2024

Should we also beta-nominate this?

Copy link
Member

@blyxyas blyxyas left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! ❤️

@blyxyas
Copy link
Member

blyxyas commented Mar 25, 2024

@bors r+

@bors
Copy link
Contributor

bors commented Mar 25, 2024

📌 Commit dafb7f6 has been approved by blyxyas

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Mar 25, 2024

⌛ Testing commit dafb7f6 with merge ca6a647...

@bors
Copy link
Contributor

bors commented Mar 25, 2024

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

1 similar comment
@bors
Copy link
Contributor

bors commented Mar 25, 2024

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

@bors bors merged commit ca6a647 into rust-lang:master Mar 25, 2024
5 checks passed
@bors
Copy link
Contributor

bors commented Mar 25, 2024

👀 Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request.

@Kobzol Kobzol deleted the assigning-clones-unspecified branch March 25, 2024 21:49
@Kobzol
Copy link
Contributor Author

Kobzol commented Mar 25, 2024

Should we also beta-nominate this?

Oh, did beta branch with this? In that case probably yes.

@blyxyas
Copy link
Member

blyxyas commented Mar 25, 2024

The lint was released on 1.77, released on March. It hasn't been two weeks since March 21. No need for nominating :)

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.

4 participants