-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Regression tests for #5201 #5207
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
@rust-highfive: 🔑 Insufficient privileges: Not in reviewers |
@rust-highfive: 🔑 Insufficient privileges: not in try users |
@bors r+ |
📌 Commit a81bb13 has been approved by |
⌛ Testing commit a81bb138e16232a4c25557ed6dc2ef1ab8c527eb with merge 2808d886228252c29d1fdda242c6e7781fdfc532... |
👍 |
@bors r- Appveyor failure is legit :sigh: |
Better safe than sorry!
@bors r+ Turns out, we already have a tests, which sort-of exposes this behavior, by giving clap's error message instead of ours :) Amended this test instead of creating a new one. |
📌 Commit a1735c7 has been approved by |
@bors: p=1 |
@bors: p=2 |
Regression tests for #5201 Better safe than sorry!
☀️ Test successful - status-appveyor, status-travis |
Better safe than sorry!