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

Metadata drivers #73

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Metadata drivers #73

wants to merge 4 commits into from

Conversation

guiwoda
Copy link
Contributor

@guiwoda guiwoda commented Dec 17, 2014

Adds support for all current doctrine metadata drivers through a new configuration key.

@soyuka
Copy link
Contributor

soyuka commented Jan 28, 2015

This is awesome, why isn't it merged? I've seen that there is a configuration change but it could be pushed on a new minor version.

@matthisstenius
Copy link

+1

@guiwoda
Copy link
Contributor Author

guiwoda commented Jan 29, 2015

@mitchellvanw let me know if there's anything I can do to help push this change forward!

@rgomezcasas
Copy link

Any plans to merge this?

@kirkbushell
Copy link
Collaborator

We can't merge just yet as we're working on the new version to support L5.

@guiwoda thanks heaps for this. We're currently discussing this internally as to how we want to support the drivers, and will most certainly use this as a study case.

That's not to say it won't get merged - it's just a matter of timing and whether it fits with @mitchellvanw's ideas.

@rgomezcasas
Copy link

Perfect! If you need any help let me know!

@guiwoda
Copy link
Contributor Author

guiwoda commented Mar 5, 2015

@kirkbushell glad it helped!
I had the intention of doing a more Laravelish metadata driver. Right now I'm using half of this package's objects and half of my own because of this. Hope you guys can figure it out!

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

Successfully merging this pull request may close these issues.

5 participants