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

enabled Symfony 3.0 support #492

Merged
merged 1 commit into from
Nov 10, 2015
Merged

Conversation

lsmith77
Copy link
Contributor

@lsmith77 lsmith77 commented Nov 6, 2015

also tweaked some aspects of the travis configuration and the composer requires

@lsmith77 lsmith77 force-pushed the symfony_3_0 branch 4 times, most recently from 1dc70a1 to 1f00cea Compare November 6, 2015 22:12
"jms/serializer": "^1.0.0",
"symfony/framework-bundle": "~2.1"
"phpoption/phpoption": "^1.0.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this probably should have been in the jms/serializer

"jms/serializer": "^1.0.0",
"symfony/framework-bundle": "~2.1"
"phpoption/phpoption": "^1.1.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this probably should also be set jms/serializer but it needs to be here as well or we have to bump the minimal version of jms/serializer to a version that includes this requirement.

@lsmith77
Copy link
Contributor Author

lsmith77 commented Nov 6, 2015

@schmittjoh would be nice to merge this and tag 1.1 :)

@drgomesp
Copy link

drgomesp commented Nov 9, 2015

@schmittjoh would love that to happen asap 👍

@weaverryan
Copy link

👍 it's important to have this merged + a tag so that we can update other dependent bundles (e.g. FOSRestBundle) for 3.0 compat. It's a big domino effect.

schmittjoh added a commit that referenced this pull request Nov 10, 2015
@schmittjoh schmittjoh merged commit 3e396c9 into schmittjoh:master Nov 10, 2015
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.

4 participants