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

Removed boolean constraint from proxy and hydrator generation #159

Merged
merged 2 commits into from Oct 4, 2016

Conversation

Bilge
Copy link
Contributor

@Bilge Bilge commented Mar 14, 2016

Proxies and hydrators can have one of many implementation strategies as defined in Doctrine\ODM\MongoDB\Configuration and Doctrine\Common\Proxy\AbstractProxyFactory.

@Bilge
Copy link
Contributor Author

Bilge commented Mar 14, 2016

Some tests seem to have failed for unrelated reasons. I think your build is broken for 5.5 and 5.6.

@Bilge
Copy link
Contributor Author

Bilge commented Jul 5, 2016

Are any of you clowns actually going to merge this?

@malarzm
Copy link
Member

malarzm commented Jul 5, 2016

Are any of you clowns actually going to merge this?

This is exactly kind of attitude you need to demonstrate to make any maintainer merge your PRs faster.

@gianarb would you have some time to take a look?

@gianarb
Copy link
Contributor

gianarb commented Jul 6, 2016

Good catch I look in this module soon 👍

@gianarb
Copy link
Contributor

gianarb commented Oct 3, 2016

Sorry can you rebase? I will merge asap
Thanks

Proxies and hydrators can have one of many implementation strategies as defined in `Doctrine\ODM\MongoDB\Configuration` and `Doctrine\Common\Proxy\AbstractProxyFactory`.
@Bilge
Copy link
Contributor Author

Bilge commented Oct 3, 2016

@gianarb Done.

@gianarb
Copy link
Contributor

gianarb commented Oct 4, 2016

Thanks

@gianarb gianarb merged commit 749ac28 into doctrine:master Oct 4, 2016
@Bilge Bilge deleted the patch-2 branch October 6, 2016 13:39
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.

None yet

3 participants