-
Notifications
You must be signed in to change notification settings - Fork 432
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
Update ServiceContainer.php #235
Conversation
Relevant issue in parent repo: jonnnnyw#216 Declaration of JonnyW\..\ServiceContainer::load() should be compatible with Symfony\..\Container::load($file)
+1 |
Please merge this |
Fix as per jonnnnyw#235
composer.json
Outdated
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "jonnyw/php-phantomjs", | |||
"description": "A PHP wrapper for loading pages through PhantomJS", | |||
"name": "cocostru/php-phantomjs", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should remove this and similar modifications in order to consider merge of this PR.
composer.json
Outdated
@@ -9,6 +9,11 @@ | |||
"name": "Jonny Wenmoth", | |||
"email": "[email protected]", | |||
"homepage": "http://jonnyw.me/" | |||
}, | |||
{ | |||
"name": "Denis Kostukovic", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be removed before merging to the main repository.
Please merge this. php-phantomjs is broken without it. |
I've sent a tweet to the author to see if he'll merge this. |
#216
#217
#222
#225
#231
#232
#247
#259
#275