Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Suppress tput stderr. Use COLUMNS env var 1st if available.#113

Merged
danielbachhuber merged 1 commit into
masterwp-cli/php-cli-tools:masterfrom
issue_112wp-cli/php-cli-tools:issue_112Copy head branch name to clipboard
Jul 26, 2017
Merged

Suppress tput stderr. Use COLUMNS env var 1st if available.#113
danielbachhuber merged 1 commit into
masterwp-cli/php-cli-tools:masterfrom
issue_112wp-cli/php-cli-tools:issue_112Copy head branch name to clipboard

Conversation

@gitlost

@gitlost gitlost commented Jul 26, 2017

Copy link
Copy Markdown
Contributor

Fixes #112

In Shell::columns() uses the COLUMNS environment variable first if available, and suppresses any errors on invoking tput by redirecting to /dev/null.

Didn't bother with trying stty as COLUMNS should basically always work I think.

@danielbachhuber danielbachhuber added this to the 0.11.5 milestone Jul 26, 2017
@danielbachhuber danielbachhuber merged commit c5f9c03 into master Jul 26, 2017
@danielbachhuber danielbachhuber deleted the issue_112 branch July 26, 2017 18:17
@danielbachhuber

Copy link
Copy Markdown
Member

Thanks again, @gitlost. I've published v0.11.5

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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.