Skip to content

added consistency to the documentation with --show-version showing the r... #17

Merged
merged 1 commit into from Dec 18, 2013

2 participants

@cordoval
cordoval commented Jan 5, 2012

...ight output with versions displaying and migrating to specific version using the data format argument

@cordoval cordoval added consistency to the documentation with --show-version showing th…
…e right output with versions displaying and migrating to specific version using the data format argument
9f513fd
@odino
Doctrine member
odino commented Dec 9, 2013

hey @cordoval is this still on?

@cordoval
cordoval commented Dec 9, 2013

@odino could you please elaborate?

@odino
Doctrine member
odino commented Dec 9, 2013

Im asking if the issue is still there :)

@odino odino commented on the diff Dec 18, 2013
Resources/doc/index.rst
@@ -158,7 +158,7 @@ finally migrate when you're ready:
.. code-block:: bash
- php app/console doctrine:migrations:migrate
+ php app/console doctrine:migrations:migrate 20100621140655
@odino
Doctrine member
odino added a note Dec 18, 2013

the problem here is that people would get a hint of executing each migration one by one, whereas the recommended way is (my guess) to just run doctrine:migrations:migrate

What do you think?

@cordoval
cordoval added a note Dec 18, 2013

oh i see they made you maintainer of the bundle, never knew.

No, I disagree. The thing here is preparing ground work for what is below the up and downs. I think most people will know about migrate but not the other so it actually encourage good practice. To be careful.

@odino
Doctrine member
odino added a note Dec 18, 2013

I see doctrine:migrations:migrate is also mentioned elsewhere, so I guess this can be merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@odino odino merged commit 62e20bb into doctrine:master Dec 18, 2013
@cordoval cordoval deleted the cordoval:patch-1 branch Dec 18, 2013
@cordoval

i feel relieved after 2 years. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.