Skip to content
This repository has been archived by the owner on May 1, 2019. It is now read-only.

Enhancement: Use doctrine/migrations to modify the database schema #502

Merged
merged 11 commits into from
Oct 19, 2015

Conversation

localheinz
Copy link
Member

This PR

  • requires doctrine/doctrine-module
  • requires doctrine/doctrine-orm-module
  • requires doctrine/migrations
  • adds a basic configuration for the DoctrineORMModule (minimum required for migrations)
  • enables both DoctrineModule and DoctrineORMModule
  • extracts a migration from data/sql/0.sql
  • uses migrations to set up database in tests
  • removes the database dump
  • adds a test case for migrations
  • moves migrations into Application namespace
  • updates build target to run migrations in production, too

Related to #497.

💁 The Doctrine console commands are available via

$ php public/index.php

Enjoy!

@localheinz localheinz self-assigned this Sep 26, 2015
@localheinz localheinz force-pushed the feature/doctrine-migrations branch 2 times, most recently from d54ead9 to cabda7f Compare September 26, 2015 11:00
@localheinz localheinz assigned Ocramius and unassigned localheinz Sep 26, 2015
@localheinz
Copy link
Member Author

@ins0 @Ocramius

Have a look at this! 😃

@localheinz localheinz mentioned this pull request Sep 26, 2015
8 tasks
@localheinz localheinz force-pushed the feature/doctrine-migrations branch 3 times, most recently from 51045e3 to 0568635 Compare September 26, 2015 19:49
@gianarb
Copy link
Contributor

gianarb commented Oct 17, 2015

This is a hight priority respect #497

And it is ready to go!! :)

@Ocramius
Copy link
Member

Needs a rebase, sadly :-(

@gianarb
Copy link
Contributor

gianarb commented Oct 18, 2015

bad day

@localheinz
Copy link
Member Author

@gianarb @Ocramius

Done! ☺️

@gianarb
Copy link
Contributor

gianarb commented Oct 18, 2015

wowowo!!!! 🎈

Ocramius added a commit that referenced this pull request Oct 19, 2015
Enhancement: Use doctrine/migrations to modify the database schema
@Ocramius Ocramius merged commit db88071 into zendframework:master Oct 19, 2015
@Ocramius
Copy link
Member

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants