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

Add tests on fixed ICEs #47044

Merged
merged 1 commit into from
Dec 31, 2017
Merged

Add tests on fixed ICEs #47044

merged 1 commit into from
Dec 31, 2017

Conversation

topecongiro
Copy link
Contributor

@topecongiro topecongiro commented Dec 28, 2017

Closes #29924. Closes #38857. Closes #39665. Closes #39872.
Closes #41210. Closes #41880. Closes #43483.

Note that compile-fail/E0599.rs is for #41210.

@rust-highfive
Copy link
Collaborator

r? @aturon

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

}

impl Trait {
//~^ ERROR E0038
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you use actual error messages instead of codes outside of E0___.rs tests?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for your review. Updated and rebased.

Copy link
Contributor

Choose a reason for hiding this comment

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

@topecongiro could you do that change in all compile-fail files?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@estebank Update tests and reabased. Also fixed the issue number (#39953 -> #39553).

@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 28, 2017
@estebank
Copy link
Contributor

#39953 is a PR, removed from description.

@estebank
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Dec 31, 2017

📌 Commit b3c022d has been approved by estebank

@bors
Copy link
Contributor

bors commented Dec 31, 2017

⌛ Testing commit b3c022d with merge 2d74ce2...

bors added a commit that referenced this pull request Dec 31, 2017
Add tests on fixed ICEs

Closes #29924. Closes #38857. Closes #39665. Closes #39872.
Closes #41210. Closes #41880. Closes #43483.

Note that compile-fail/E0599.rs is for #41210.
@bors
Copy link
Contributor

bors commented Dec 31, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: estebank
Pushing 2d74ce2 to master...

@bors bors merged commit b3c022d into rust-lang:master Dec 31, 2017
@topecongiro topecongiro deleted the fixed-ices branch December 31, 2017 07:31
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.

7 participants