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

added boot method to the DoctrineORMServiceProvider #16

Merged
merged 1 commit into from
Feb 18, 2020

Conversation

gpunx
Copy link
Contributor

@gpunx gpunx commented Jun 17, 2012

added the boot method to comply with ServiceProviderInterface. the lack of the boot method was causing an error in the latest version of silex.

…ack of the boot method was causing an error in the latest version of silex.
@itavero
Copy link
Contributor

itavero commented Jul 9, 2012

It's been a while since I've used this service provider but I just noticed this issue too.
Hopefully @mjakubowski can have a look at it soon!

@niepi
Copy link

niepi commented Jul 12, 2012

+1 Hopefully @mjakubowski can have a look at it soon!

works for me, but a merge would be nice, so we can use this with composer

@hisem
Copy link

hisem commented Aug 10, 2012

+1 Hopefully @mjakubowski can have a look at it soon!

@kleingeist
Copy link

+1 and ping @mjakubowski :
i hope you'll update the ServiceProvider soon. if you dont have the time to mantain the extension maybe you could find some additional maintainers? i guess there are some people around interested in helping

@niepi
Copy link

niepi commented Aug 23, 2012

i don't think anything is happening here, so maybe someone should fork it and change the link in the silex wiki

@itavero
Copy link
Contributor

itavero commented Aug 23, 2012

@niepi It's better if @mjakubowski would add some maintainers as suggested by @kleingeist, in my opinion.

(I believe @mjakubowski might be on vacation right now. Let's hope he'll be back soon.)

@niepi
Copy link

niepi commented Aug 23, 2012

@itavero your right, but this issue is now two months old.

@antonmedv
Copy link

Come on! Fix it! Any forks?

@itavero itavero mentioned this pull request Oct 24, 2012
radeksimko added a commit to TimeIncUK/doctrine-orm-extension that referenced this pull request Feb 3, 2014
@radeksimko
Copy link

You can use your own repo or this repo:
https://github.com/ipcmedia/doctrine-orm-extension

Just update your composer.json with VCS source or use directly the one I've just uploaded to Packagist. The v1.1.0 version should have this bugfix.

radeksimko added a commit to TimeIncUK/doctrine-orm-extension that referenced this pull request Feb 3, 2014
@mjakubowski mjakubowski merged commit e124edd into mjakubowski:master Feb 18, 2020
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

8 participants