-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Fix tests that used // | help:
annotations
#30773
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
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
Hm out of curiosity, these are actually tested, right? I vaguely remember that at some point in the past these may not have been checked (specifically help, not other errors). |
Yep, they are checked. |
⌛ Testing commit 6ce9ac4 with merge b035da4... |
💔 Test failed - auto-linux-64-opt |
@bors: retry On Thu, Jan 14, 2016 at 8:47 PM, bors notifications@github.com wrote:
|
In my PR for #21659 I accidentally used `// | help` as test annotation. This PR updates it to `//~| help`. I also found and updated 2 other tests with the same issue.
💔 Test failed - auto-linux-64-nopt-t |
@bors: retry On Fri, Jan 15, 2016 at 2:04 AM, bors notifications@github.com wrote:
|
In my PR for #21659 I accidentally used `// | help` as test annotation. This PR updates it to `//~| help`. I also found and updated 2 other tests with the same issue.
In my PR for #21659 I accidentally used
// | help
as test annotation. This PR updates it to//~| help
. I also found and updated 2 other tests with the same issue.