Skip to content

Add long diagnostics for E0015 and E0020. - #24488

Closed
nham wants to merge 2 commits into
rust-lang:masterfrom
nham:explain_check_const
Closed

Add long diagnostics for E0015 and E0020.#24488
nham wants to merge 2 commits into
rust-lang:masterfrom
nham:explain_check_const

Conversation

@nham

@nhamnham commented Apr 16, 2015

Copy link
Copy Markdown
Contributor

Part of #24407.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @brson

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

@nham
nhamforce-pushed the explain_check_const branch 2 times, most recently from 8a5238d to 62a73c0CompareApril 16, 2015 05:59
@michaelsproul

Copy link
Copy Markdown
Contributor

Looks good to me 👍

@alexcrichton

Copy link
Copy Markdown
Member

Thanks @nham! r=me with @lfairy's comment

@nham
nhamforce-pushed the explain_check_const branch from 62a73c0 to f043695CompareApril 16, 2015 17:44
@nham

nham commented Apr 16, 2015

Copy link
Copy Markdown
ContributorAuthor

Thanks, I've fixed the wording in E0015.

@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+ f043695 rollup

bors added a commit that referenced this pull request Apr 16, 2015
bors added a commit that referenced this pull request Apr 17, 2015
bors added a commit that referenced this pull request Apr 17, 2015
@bors

bors commented Apr 17, 2015

Copy link
Copy Markdown
Collaborator

☔ The latest upstream changes (presumably #24512) made this pull request unmergeable. Please resolve the merge conflicts.

@nham
nhamforce-pushed the explain_check_const branch from f043695 to 8d795efCompareApril 17, 2015 17:12
@nham

nham commented Apr 17, 2015

Copy link
Copy Markdown
ContributorAuthor

I rebased on top of master.

@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+ 8d795ef

@alexcrichton

Copy link
Copy Markdown
Member

Closing in favor of the merged version at #24542

Manishearth added a commit to Manishearth/rust that referenced this pull request Apr 18, 2015
I did a manual merge of all the extended error PRs as we were getting merge conflicts yesterday. I think this is preferable to merging separately as I ended up having to manually merge @nham and @GuillaumeGomez's commits.
Rollup of rust-lang#24458, rust-lang#24482 and rust-lang#24488.
rust-lang#24482 and rust-lang#24488 were already re-approved, and would need to be cancelled if this is merged instead.
Sign up for freeto 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.

6 participants

@nham@rust-highfive@michaelsproul@alexcrichton@bors@brson