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

Fix note for type alias in trait position #32005

Merged
merged 4 commits into from
Mar 3, 2016
Merged

Fix note for type alias in trait position #32005

merged 4 commits into from
Mar 3, 2016

Conversation

vegai
Copy link
Contributor

@vegai vegai commented Mar 2, 2016

Fix note for type alias in trait position

Fixes #31686

@rust-highfive
Copy link
Collaborator

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nikomatsakis (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

@Manishearth
Copy link
Member

You'll need to update the tests too. tests/compile-fail/issue-5035.rs will need a //~^^ NOTE: marker with the second note

You can check the test locally by running make check-stage1-cfail TESTNAME=issue-5035

Let a complete compile-fail run go through as well (make check-stage1-cfail) to ensure that other tests aren't affected.

@vegai
Copy link
Contributor Author

vegai commented Mar 2, 2016

Yep, also issue-3907.rs broke. Fixed'em both.

@Manishearth
Copy link
Member

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented Mar 2, 2016

📌 Commit 01a6e86 has been approved by Manishearth

@bluss bluss changed the title Fix for https://github.com/rust-lang/rust/issues/31686 Fix note for type alias in trait position Mar 2, 2016
@bluss
Copy link
Member

bluss commented Mar 2, 2016

I edited the PR title and the PR message, so that a small description of the change lands in the merge message.

@bors
Copy link
Contributor

bors commented Mar 2, 2016

⌛ Testing commit 01a6e86 with merge 7b0b80a...

bors added a commit that referenced this pull request Mar 2, 2016
Fix note for type alias in trait position

Fixes #31686
@bors bors merged commit 01a6e86 into rust-lang:master Mar 3, 2016
@vegai vegai deleted the 31686 branch March 3, 2016 07:01
Sign up for free to 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