Skip to content

test: Move missing-items to a ui test #37771

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

Merged
merged 1 commit into from
Nov 15, 2016

Conversation

alexcrichton
Copy link
Member

This test is failing on nightly for unknown reasons, and my best guess is a
difference in grep versions which is interpreting symbols differently. For now
let's just move this to a ui test and hope it fixes nightlies.

This test is failing on nightly for unknown reasons, and my best guess is a
difference in grep versions which is interpreting symbols differently. For now
let's just move this to a ui test and hope it fixes nightlies.
@rust-highfive
Copy link
Contributor

r? @brson

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

@brson
Copy link
Contributor

brson commented Nov 14, 2016

@bors r+ p=1

@bors
Copy link
Collaborator

bors commented Nov 14, 2016

📌 Commit 3216024 has been approved by brson

@Arnavion
Copy link

The | ^ bit needs to be properly escaped, atleast. Not sure if that's the reason.

@bors
Copy link
Collaborator

bors commented Nov 14, 2016

⌛ Testing commit 3216024 with merge bd6a54d...

bors added a commit that referenced this pull request Nov 14, 2016
test: Move missing-items to a ui test

This test is failing on nightly for unknown reasons, and my best guess is a
difference in grep versions which is interpreting symbols differently. For now
let's just move this to a ui test and hope it fixes nightlies.
@bors bors merged commit 3216024 into rust-lang:master Nov 15, 2016
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.

5 participants