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

Improve expl_impl_clone_on_copy #6993

Merged
merged 1 commit into from
Mar 28, 2021
Merged

Conversation

Jarcho
Copy link
Contributor

@Jarcho Jarcho commented Mar 28, 2021

fixes: #1254

changelog: Check to see if the generic constraints are the same as if using derive for expl_impl_clone_on_copy

@rust-highfive
Copy link

r? @llogiq

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Mar 28, 2021
@llogiq
Copy link
Contributor

llogiq commented Mar 28, 2021

You're really on a roll. 😀 Thank you!

@bors r+

@bors
Copy link
Collaborator

bors commented Mar 28, 2021

📌 Commit 5b2fb8c has been approved by llogiq

@bors
Copy link
Collaborator

bors commented Mar 28, 2021

⌛ Testing commit 5b2fb8c with merge 09a57cf...

bors added a commit that referenced this pull request Mar 28, 2021
Improve `expl_impl_clone_on_copy`

fixes: #1254

changelog: Check to see if the generic constraints are the same as if using derive for `expl_impl_clone_on_copy`
@bors
Copy link
Collaborator

bors commented Mar 28, 2021

💔 Test failed - checks-action_dev_test

Check to see if the generic constraints are the same as if using derive
@llogiq
Copy link
Contributor

llogiq commented Mar 28, 2021

@bors r+

@bors
Copy link
Collaborator

bors commented Mar 28, 2021

📌 Commit 879fa5c has been approved by llogiq

@bors
Copy link
Collaborator

bors commented Mar 28, 2021

⌛ Testing commit 879fa5c with merge c07103b...

@bors
Copy link
Collaborator

bors commented Mar 28, 2021

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

@bors bors merged commit c07103b into rust-lang:master Mar 28, 2021
@Jarcho Jarcho deleted the expl_impl_clone branch April 6, 2021 16:08
bors added a commit that referenced this pull request Aug 17, 2021
Update docs for `expl_impl_clone_on_copy`

The known issue was fixed in #6993. I missed updating the docs then, so it's happening now.

changelog: None
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.

Disable expl_impl_clone_on_copy when Copy is manually implemented.
4 participants