-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
[FrameworkBundle] Applied new styles to the config:debug & config:dump-reference commands #14591
Conversation
@ogizanagi Can you please show before and after screenshots? |
if (class_exists('Symfony\Component\Console\Helper\Table')) { | ||
$table->render(); | ||
$message = 'Available registered bundles with their extension alias if available:'; | ||
if ($output instanceof OutputStyle) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this check for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This method is typehinted OutputInterface
. As it might eventually be used by other bundles, not using SymfonyStyle nor custom StyleInterface
implementations, we need to do this check in order to choose the way we'll use in order to render the table.
Unless we judge other bundles should not extend this class, so we could typehint this method with the StyleInterface
.
BTW, I don't remember why I used OutputStyle
instead of StyleInterface
:/
…p-reference commands
👍 |
Thank you @ogizanagi. |
… & config:dump-reference commands (ogizanagi) This PR was merged into the 2.7 branch. Discussion ---------- [FrameworkBundle] Applied new styles to the config:debug & config:dump-reference commands | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | Partially #14138 | License | MIT | Doc PR | - Min dependency on `symfony/console` is changed to `~2.7` in the FrameworkBundle, as it will be a requirement in order to update each command provided in this bundle. Commits ------- e56a619 [FrameworkBundle] Applied new styles to the config:debug & config:dump-reference commands
Min dependency on
symfony/console
is changed to~2.7
in the FrameworkBundle, as it will be a requirement in order to update each command provided in this bundle.