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.

Sign up for GitHub

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

Jump to bottom

Add tests for #17001, #21449, #22992, #23208, #23442 #28299

Merged
merged 1 commit into from
Sep 9, 2015
Merged

Add tests for #17001, #21449, #22992, #23208, #23442 #28299

merged 1 commit into from
Sep 9, 2015

Conversation

apasel422
Copy link
Contributor

Closes #17001
Closes #21449
Closes #22992
Closes #23208
Closes #23442

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

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

@alexcrichton
Copy link
Member

@bors: r+ 6a66749b9573a5dd32b0a1d818ecad9b9e2a31ba

Nice!

@apasel422
Copy link
Contributor Author

@alexcrichton pretty failed on issue-22992.rs. How should I proceed?

@alexcrichton
Copy link
Member

Feel free to just throw // ignore-pretty in the test

@apasel422
Copy link
Contributor Author

@alexcrichton Done. Tests passed.

@alexcrichton
Copy link
Member

@bors: r+ 04fff50

@bors
Copy link
Contributor

bors commented Sep 9, 2015

⌛ Testing commit 04fff50 with merge eaa5bf3...

bors added a commit that referenced this pull request Sep 9, 2015
@bors
Auto merge of #28299 - apasel422:tests, r=alexcrichton
eaa5bf3 
Closes #17001
Closes #21449
Closes #22992
Closes #23208
Closes #23442
@bors
Copy link
Contributor

bors commented Sep 9, 2015

☀️ Test successful - auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-x-android-t, auto-mac-32-opt, auto-mac-64-nopt-t, auto-mac-64-opt, auto-win-gnu-32-nopt-t, auto-win-gnu-32-opt, auto-win-gnu-64-nopt-t, auto-win-gnu-64-opt, auto-win-msvc-32-opt, auto-win-msvc-64-opt

@bors bors merged commit 04fff50 into rust-lang:master Sep 9, 2015
@apasel422 apasel422 deleted the tests branch September 10, 2015 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nikomatsakis nikomatsakis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@apasel422 @rust-highfive @alexcrichton @bors @nikomatsakis