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

Allow for limiting detected term width #654

Merged
merged 1 commit into from
Sep 10, 2016
Merged

Allow for limiting detected term width #654

merged 1 commit into from
Sep 10, 2016

Conversation

nabijaczleweli
Copy link
Contributor

Closes #653

@yo-bot
Copy link

yo-bot commented Sep 10, 2016

r? @Vinatorul

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

@nabijaczleweli
Copy link
Contributor Author

nabijaczleweli commented Sep 10, 2016

@kbknapp Is this what you had in mind?

@yo-bot yo-bot assigned kbknapp and unassigned Vinatorul Sep 10, 2016
@kbknapp
Copy link
Member

kbknapp commented Sep 10, 2016

Yep, exactly! If you can rebase away the WIP messages this should be good.

@nabijaczleweli nabijaczleweli changed the title [WIP] Allow for limiting detected term width Allow for limiting detected term width Sep 10, 2016
Useful when, e.g., the terminal is fullscreen

Closes #653
@nabijaczleweli
Copy link
Contributor Author

Squashed

@kbknapp
Copy link
Member

kbknapp commented Sep 10, 2016

@homu r+

@homu
Copy link
Contributor

homu commented Sep 10, 2016

📌 Commit a43e28a has been approved by kbknapp

homu added a commit that referenced this pull request Sep 10, 2016
… r=kbknapp

Allow for limiting detected term width

Closes #653
@homu
Copy link
Contributor

homu commented Sep 10, 2016

⌛ Testing commit a43e28a with merge 3834063...

homu added a commit that referenced this pull request Sep 10, 2016
… r=kbknapp

Allow for limiting detected term width

Closes #653
@homu
Copy link
Contributor

homu commented Sep 10, 2016

☀️ Test successful - status

@homu homu merged commit a43e28a into clap-rs:master Sep 10, 2016
@nabijaczleweli nabijaczleweli deleted the feature/help/term-width-override branch September 10, 2016 20:05
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