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

V2.4.0 #493

Merged
merged 8 commits into from May 4, 2016
Merged

V2.4.0 #493

merged 8 commits into from May 4, 2016

Conversation

kbknapp
Copy link
Member

@kbknapp kbknapp commented May 3, 2016

No description provided.

@yo-bot
Copy link

yo-bot commented May 3, 2016

r? @Vinatorul

(yo-bot has picked a reviewer for you, use r? to override)

…rgs are present

Adds three new methods of `Arg` which allow for specifying three new
types of rules.

* `Arg::required_unless`

Allows saying a particular arg is only required if a specific other arg
*isn't* present.

* `Arg::required_unless_all`

Allows saying a particular arg is only required so long as *all* the
following args aren't present

* `Arg::required_unless_one`

Allows saying a particular arg is required unless at least one of the
following args is present.
Can now use the `App::before_help` method to add additional information
that will be displayed prior to the help message. Common uses are
copyright, or license information.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 87.232% when pulling 5b01200 on v2.4.0 into 84381b5 on master.

@kbknapp
Copy link
Member Author

kbknapp commented May 3, 2016

@homu r+

@homu
Copy link
Contributor

homu commented May 3, 2016

📌 Commit 5b01200 has been approved by kbknapp

homu added a commit that referenced this pull request May 3, 2016
homu added a commit that referenced this pull request May 3, 2016
@homu
Copy link
Contributor

homu commented May 3, 2016

⌛ Testing commit 5b01200 with merge 841cbf5...

@homu
Copy link
Contributor

homu commented May 3, 2016

☀️ Test successful - status

@homu
Copy link
Contributor

homu commented May 3, 2016

👀 Test was successful, but fast-forwarding failed: 422 Required status check "coverage/coveralls" is pending

@kbknapp
Copy link
Member Author

kbknapp commented May 3, 2016

@homu retry

homu added a commit that referenced this pull request May 3, 2016
homu added a commit that referenced this pull request May 3, 2016
@homu
Copy link
Contributor

homu commented May 3, 2016

⌛ Testing commit 5b01200 with merge 630d681...

@homu
Copy link
Contributor

homu commented May 3, 2016

☀️ Test successful - status

@homu
Copy link
Contributor

homu commented May 3, 2016

👀 Test was successful, but fast-forwarding failed: 422 Required status check "coverage/coveralls" is pending

@kbknapp kbknapp merged commit dae2d93 into master May 4, 2016
@kbknapp kbknapp deleted the v2.4.0 branch May 4, 2016 01:09
epage added a commit to epage/clap that referenced this pull request Oct 6, 2021
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.

None yet

5 participants