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.

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

Display more informative messages for migration aliases #978

Merged
merged 3 commits into from
May 23, 2020

Conversation

goetas
Copy link
Member

@goetas goetas commented May 15, 2020

Q A
Type improvement
BC Break -
Fixed issues -

This PR does not perform any major functional change, it mainly adds some formatting to the error/warning when there are no migrations to migrate.

  • 812a3c2 does some minor code-improvements by simplifying how error messages are printed and aliases are resolved
  • 6592e88 displays correctly the latest migration on the status command
  • 069eea0 allows to resolve the migration 0 as first migration)

@goetas goetas added this to the 3.0.0 milestone May 15, 2020
@goetas goetas force-pushed the already-latest-no-error branch 2 times, most recently from f4d7052 to 491e07e Compare May 22, 2020 07:53
@goetas goetas changed the title change alias resolution Display more informative messages for migration aliases May 22, 2020
@goetas goetas marked this pull request as ready for review May 22, 2020 07:59
@goetas goetas requested a review from greg0ire May 22, 2020 10:29
@goetas goetas merged commit af6a764 into doctrine:master May 23, 2020
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.

None yet

2 participants