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

Regression tests for #5201 #5207

Merged
merged 1 commit into from
Mar 19, 2018
Merged

Regression tests for #5201 #5207

merged 1 commit into from
Mar 19, 2018

Conversation

matklad
Copy link
Member

@matklad matklad commented Mar 19, 2018

Better safe than sorry!

@rust-highfive
Copy link

r? @alexcrichton

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

@bors
Copy link
Collaborator

bors commented Mar 19, 2018

@rust-highfive: 🔑 Insufficient privileges: Not in reviewers

@bors
Copy link
Collaborator

bors commented Mar 19, 2018

@rust-highfive: 🔑 Insufficient privileges: not in try users

@matklad
Copy link
Member Author

matklad commented Mar 19, 2018

@bors r+

@bors
Copy link
Collaborator

bors commented Mar 19, 2018

📌 Commit a81bb13 has been approved by matklad

@bors
Copy link
Collaborator

bors commented Mar 19, 2018

⌛ Testing commit a81bb138e16232a4c25557ed6dc2ef1ab8c527eb with merge 2808d886228252c29d1fdda242c6e7781fdfc532...

@alexcrichton
Copy link
Member

👍

@matklad
Copy link
Member Author

matklad commented Mar 19, 2018

@bors r-

Appveyor failure is legit :sigh:

Better safe than sorry!
@matklad
Copy link
Member Author

matklad commented Mar 19, 2018

@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.

@bors
Copy link
Collaborator

bors commented Mar 19, 2018

📌 Commit a1735c7 has been approved by matklad

@alexcrichton
Copy link
Member

@bors: p=1

@matklad
Copy link
Member Author

matklad commented Mar 19, 2018

@bors: p=2

@bors
Copy link
Collaborator

bors commented Mar 19, 2018

⌛ Testing commit a1735c7 with merge 2aba2d9...

bors added a commit that referenced this pull request Mar 19, 2018
Regression tests for #5201

Better safe than sorry!
@bors
Copy link
Collaborator

bors commented Mar 19, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: matklad
Pushing 2aba2d9 to master...

@bors bors merged commit a1735c7 into rust-lang:master Mar 19, 2018
@matklad matklad deleted the clap-test branch March 20, 2018 07:51
@ehuss ehuss added this to the 1.26.0 milestone Feb 6, 2022
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