Include filter expressions into diff #66

Merged
merged 2 commits into from Jul 25, 2012

Projects

None yet

3 participants

@Hounddog

Included command line --filter-expression to include/exclude tables by Regular Expression

@stof stof and 1 other commented on an outdated diff Apr 17, 2012
...DBAL/Migrations/Tools/Console/Command/DiffCommand.php
@@ -53,8 +53,8 @@ protected function configure()
<info>%command.full_name% --editor-cmd=mate</info>
EOT
- );
-
+ )
+ ->addOption('filter-expression', null, InputOption::VALUE_OPTIONAL, 'Tables which are filtered by Regular Expression.');
stof
stof Apr 17, 2012 Member

the indentation has one extra level here

stof
stof Apr 17, 2012 Member

ah no, it is a missing level for the previous closing brace

Hounddog
Hounddog Apr 17, 2012

I saw this but did not want to change anything there. if you would like i can also fix this indention

stof
stof Apr 17, 2012 Member

please fix it as the line is already part of the diff anyway as you removed the semicolon in it

Martin Shwalbe Fixed Indention b8dcff7
@beberlei beberlei merged commit b86b4cc into doctrine:master Jul 25, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment