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

Issues rollup #637

Merged
merged 10 commits into from
Aug 28, 2016
Merged

Issues rollup #637

merged 10 commits into from
Aug 28, 2016

Conversation

kbknapp
Copy link
Member

@kbknapp kbknapp commented Aug 28, 2016

No description provided.

… crate_version! macros

One can now use

```rust
let a = App::with_defaults("My Program");

// same as
let a2 = App::new("My Program")
	.version(crate_version!())
	.author(crate_authors!());
```

Closes #600
When errors are made while developing, the panic error messages have
been improved instead of relying on the default panic message which is
extremely unhelpful.

Closes #574
…hen term width is determined to be too small, or help text is too long

Now `clap` will check if it should automatically place long help
messages on the next line after the flag/option. This is determined by
checking to see if the space taken by flag/option plus spaces and values
doesn't leave enough room for the entirety of the help message, with the
single exception of of if the flag/option/spaces/values is less than 25%
of the width.

Closes #597
…s and ones that conflict with each other

Args that conflict *and* are in a group will now only display in the
group and not in the usage string itself.

Closes #616
@yo-bot
Copy link

yo-bot commented Aug 28, 2016

r? @sru

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

@homu
Copy link
Contributor

homu commented Aug 28, 2016

📌 Commit 063dd26 has been approved by kbknapp

homu added a commit that referenced this pull request Aug 28, 2016
homu added a commit that referenced this pull request Aug 28, 2016
@homu
Copy link
Contributor

homu commented Aug 28, 2016

⌛ Testing commit 063dd26 with merge 84dcd70...

@homu
Copy link
Contributor

homu commented Aug 28, 2016

📌 Commit a0e9a7d has been approved by kbknapp

homu added a commit that referenced this pull request Aug 28, 2016
@homu
Copy link
Contributor

homu commented Aug 28, 2016

⌛ Testing commit a0e9a7d with merge b94b1a0...

@kbknapp
Copy link
Member Author

kbknapp commented Aug 28, 2016

@homu r+

@homu
Copy link
Contributor

homu commented Aug 28, 2016

📌 Commit a0e9a7d has been approved by kbknapp

@kbknapp
Copy link
Member Author

kbknapp commented Aug 28, 2016

@homu r+

@homu
Copy link
Contributor

homu commented Aug 28, 2016

💡 This pull request was already approved, no need to approve it again.

  • There's another pull request that is currently being tested, blocking this pull request: Issues rollup #637

@homu
Copy link
Contributor

homu commented Aug 28, 2016

📌 Commit a0e9a7d has been approved by kbknapp

@homu
Copy link
Contributor

homu commented Aug 28, 2016

💡 This pull request was already approved, no need to approve it again.

  • There's another pull request that is currently being tested, blocking this pull request: Issues rollup #637

@kbknapp
Copy link
Member Author

kbknapp commented Aug 28, 2016

@homu r-

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.

4 participants