Update compatibility note for 1.28.0 to be correct by Havvy · Pull Request #52929 · rust-lang/rust (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation4 Commits1 Checks0 Files changed

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

Havvy

You can still put implementations on dyn Trait + Send + Send, but it'd be the same as putting them on dyn Trait + Send. This is why the error is that there are duplicate definitions in the example.

@Havvy

@rust-highfive

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton

@bors

📌 Commit bcab7e2 has been approved by alexcrichton

@bors bors added S-waiting-on-bors

Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

and removed S-waiting-on-review

Status: Awaiting review from the assignee but also interested parties.

labels

Aug 1, 2018

@Mark-Simulacrum

Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request

Aug 1, 2018

@Mark-Simulacrum

Update compatibility note for 1.28.0 to be correct

You can still put implementations on dyn Trait + Send + Send, but it'd be the same as putting them on dyn Trait + Send. This is why the error is that there are duplicate definitions in the example.

bors added a commit that referenced this pull request

Aug 1, 2018

@bors

Rollup of 31 pull requests

Successful merges:

Failed merges:

r? @ghost

@pietroalbini pietroalbini added beta-nominated

Nominated for backporting to the compiler in the beta channel.

beta-accepted

Accepted for backporting to the compiler in the beta channel.

T-release

Relevant to the release subteam, which will review and decide on the PR/issue.

labels

Aug 1, 2018

pietroalbini added a commit to pietroalbini/rust that referenced this pull request

Aug 1, 2018

@pietroalbini

Update compatibility note for 1.28.0 to be correct

You can still put implementations on dyn Trait + Send + Send, but it'd be the same as putting them on dyn Trait + Send. This is why the error is that there are duplicate definitions in the example.

bors added a commit that referenced this pull request

Aug 1, 2018

@bors

Rollup of 30 pull requests

Successful merges:

Failed merges:

r? @ghost

bors added a commit that referenced this pull request

Aug 5, 2018

@bors

[beta] Rollup backports & switch bootstrap compiler

This PR switches the bootstrap compiler from dev-static to static, and rollups some backports.

Merged and approved:

r? @Mark-Simulacrum

Labels

beta-accepted

Accepted for backporting to the compiler in the beta channel.

S-waiting-on-bors

Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

T-release

Relevant to the release subteam, which will review and decide on the PR/issue.