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 this component to work with symfony3 #287

Merged
merged 2 commits into from
Nov 9, 2015
Merged

Conversation

benji07
Copy link
Contributor

@benji07 benji07 commented Nov 6, 2015

No description provided.

- php: 5.6
env: DEPS=dev
- php: 5.6
env: DEPS=dev SYMFONY_VERSION='2.8.*'
Copy link
Contributor

Choose a reason for hiding this comment

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

2.8.*@dev would be better than a DEPS option IMO.

@tshelburne
Copy link
Collaborator

@Seldaek It's been a bit since I dealt with Symfony and PHP - does this all look good to you?

@Seldaek
Copy link
Member

Seldaek commented Nov 9, 2015

Seems fine yeah, we barely use any symfony anyway. One issue though is that AFAIK symfony3's yaml component doesn't allow unquoted @s anymore so all our @foo references will need to become '@foo', as the @ is a reserved char in the yaml spec. This might not break the tests, but I think we should update docs accordingly, and well users migrating to sf3 will have the problem but that's gonna hit them for more than alice probably (like symfony service definitions mainly).

Seldaek added a commit that referenced this pull request Nov 9, 2015
Allow this component to work with symfony3
@Seldaek Seldaek merged commit 045d59b into nelmio:master Nov 9, 2015
@soullivaneuh
Copy link
Contributor

Symfony 3.0 is now released. Did you plan a date for a new tag of this bundle? Thanks.

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