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

Unignore tests which work fine now. #41629

Merged
merged 1 commit into from May 2, 2017

Conversation

Mark-Simulacrum
Copy link
Member

As far as I can tell, these tests will now work fine. #13745 tracks the remaining tests which are ignored for various reasons.

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

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

@carols10cents carols10cents added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 1, 2017
@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented May 1, 2017

📌 Commit f48a8f5 has been approved by nikomatsakis

@bors
Copy link
Contributor

bors commented May 1, 2017

⌛ Testing commit f48a8f5 with merge de4bdd2...

bors added a commit that referenced this pull request May 1, 2017
Unignore tests which work fine now.

As far as I can tell, these tests will now work fine. #13745 tracks the remaining tests which are ignored for various reasons.
@bors
Copy link
Contributor

bors commented May 2, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: nikomatsakis
Pushing de4bdd2 to master...

@bors bors merged commit f48a8f5 into rust-lang:master May 2, 2017
@Mark-Simulacrum Mark-Simulacrum deleted the re-enable-tests branch June 22, 2017 12:35
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.

None yet

5 participants