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

allow symfony 4.0 #129

Merged
merged 2 commits into from
Nov 23, 2017
Merged

allow symfony 4.0 #129

merged 2 commits into from
Nov 23, 2017

Conversation

ElectricMaxxx
Copy link
Contributor

@ElectricMaxxx ElectricMaxxx commented Nov 20, 2017

This would allow symfony 4.0 for bundles/component requiring yours.

Pleas relase after green and merge.

@ElectricMaxxx
Copy link
Contributor Author

@jeanmonod shall i fiix the tests or are even red on master?

@dbu
Copy link
Member

dbu commented Nov 22, 2017

i restarted the master branch. this smells like it would not be related to this MR, lets see: https://travis-ci.org/liip/RMT/builds/270377916

@dbu
Copy link
Member

dbu commented Nov 22, 2017

or not. could it have something to do with allowing symfony 4? although there are old version installs, so this should not be the reason. but master is still green...

@dbu
Copy link
Member

dbu commented Nov 22, 2017

interesting, there is an open bug with exactly this message: #77. although i don't see how that would be related

@ElectricMaxxx
Copy link
Contributor Author

So lets fix it :-)

@dbu
Copy link
Member

dbu commented Nov 22, 2017

i did a minimal fix that i merged. can you please rebase your branch on master to see if the error goes away?

@dbu
Copy link
Member

dbu commented Nov 22, 2017

i also merged #131 - can you please rebase and solve conflicts?

@ElectricMaxxx
Copy link
Contributor Author

Lets see what happens.

@ElectricMaxxx
Copy link
Contributor Author

Looks green now :-)

@dbu dbu merged commit 85d993a into liip:master Nov 23, 2017
@ElectricMaxxx ElectricMaxxx deleted the allow_symfony_4.0 branch November 23, 2017 11:43
@ElectricMaxxx
Copy link
Contributor Author

@dbu can you tag a release on then? We should see an effect on: doctrine/phpcr-odm#764

@nicwortel nicwortel mentioned this pull request Dec 1, 2017
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.

2 participants