Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

javiereguiluz
Copy link
Member

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

This PR uses comment() which hasn't been merged yet. WIP PR at #15964

server_comparison_1

server_comparison_2

@javiereguiluz javiereguiluz changed the title [WCM] [Console] Updated the styles of the server commands [Console] Updated the styles of the server commands Sep 28, 2015
@fabpot
Copy link
Member

fabpot commented Sep 28, 2015

Thank you @javiereguiluz.

@fabpot fabpot closed this in d65f864 Sep 28, 2015
@@ -84,6 +85,8 @@ protected function configure()
*/
protected function execute(InputInterface $input, OutputInterface $output)
{
$stdout = $output;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is not needed. SymfonyStyle also implements OutputInterface. So it can be used just as $stdout.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nvm, see below comment

@fabpot fabpot mentioned this pull request Nov 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants