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

Support output formatter wordwraping. #57

Merged
merged 1 commit into from
Nov 18, 2016
Merged

Support output formatter wordwraping. #57

merged 1 commit into from
Nov 18, 2016

Conversation

greg-1-anderson
Copy link
Member

Disposition

This pull request:

  • Fixes a bug
  • Adds a feature
  • Breaks backwards compatibility
  • Has tests that cover functionality in this PR

Summary

Support output formatter wordwrapping.

Description

Provide a class to pass the terminal width from the Symfony Console Application object to the FormatterOptions object that needs it. Set this up via dependency injection, so that clients can alter the behavior as needed.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.3%) to 89.454% when pulling 16fb32f on wordwrap into 0bcb15e on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.3%) to 89.454% when pulling 16fb32f on wordwrap into 0bcb15e on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.2%) to 89.542% when pulling 16fb32f on wordwrap into 0bcb15e on master.

@coveralls
Copy link

coveralls commented Nov 18, 2016

Coverage Status

Coverage decreased (-1.2%) to 89.542% when pulling 2420ee8 on wordwrap into 0bcb15e on master.

@coveralls
Copy link

coveralls commented Nov 18, 2016

Coverage Status

Coverage increased (+0.1%) to 90.867% when pulling ef87a2d on wordwrap into 0bcb15e on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.06%) to 90.789% when pulling 2cc0900 on wordwrap into 0bcb15e on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.06%) to 90.789% when pulling 2cc0900 on wordwrap into 0bcb15e on master.

@coveralls
Copy link

coveralls commented Nov 18, 2016

Coverage Status

Coverage increased (+0.1%) to 90.867% when pulling 2cc0900 on wordwrap into 0bcb15e on master.

@greg-1-anderson greg-1-anderson merged commit f00f587 into master Nov 18, 2016
@dependencies dependencies bot mentioned this pull request Nov 22, 2018
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