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.

Sign up for GitHub

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

[make:*] improve output messages for Symfony CLI users #1238

Merged
merged 12 commits into from
Nov 28, 2022
Merged

[make:*] improve output messages for Symfony CLI users #1238

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
72f0ff0
[make:*] improve output messages for Symfony CLI users
jrushlow Nov 21, 2022
2aa5179
use Symfony CLI vars to determine correct command prefix
jrushlow Nov 27, 2022
d1c39d2
testcases for cli tools
jrushlow Nov 27, 2022
0a9aee3
no trailing space
jrushlow Nov 27, 2022
2552c06
guess prompt in existing makers
jrushlow Nov 27, 2022
589e8a0
ensure correct command shown in cmd output
jrushlow Nov 28, 2022
70fac1a
cleanup
jrushlow Nov 28, 2022
37ebedd
add testcases for make migration
jrushlow Nov 28, 2022
db8bbd8
use name value for prompt
jrushlow Nov 28, 2022
9e98c25
simplify tests
jrushlow Nov 28, 2022
1349524
better method name
jrushlow Nov 28, 2022
a8ec5d7
why didnt i think of this
jrushlow Nov 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions src/Maker/MakeMigration.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -122,6 +122,7 @@ public function generate(InputInterface $input, ConsoleStyle $io, Generator $gen
$io->text([
sprintf('Next: Review the new migration <info>%s</info>', $migrationName),
'Then: Run the migration with <info>php bin/console doctrine:migrations:migrate</info>',
'or if you\'re using the Symfony CLI: <info>symfony console doctrine:migrations:migrate</info>',
'See <fg=yellow>https://symfony.com/doc/current/bundles/DoctrineMigrationsBundle/index.html</>',
]);
}
Expand Down