Skip to content

Conversation

javiereguiluz
Copy link
Member

This fixes #5951.

contain BC breaks. However, Symfony tries to make this upgrade process as
smooth as possible.
Every two years, Symfony releases a new major version release (the first number
changes). These releases are the trickiest to upgrade, as they are allowed break
Copy link

Choose a reason for hiding this comment

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

allowed to break backwards compatibility

@snoek09
Copy link

snoek09 commented Sep 4, 2016

Cool stuff 👍

One minor comment.

Status: reviewed.

Copy link
Member

@wouterj wouterj left a comment

Choose a reason for hiding this comment

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

Afaik, it only works for Symfony 2 to 3 atm, I think we have to mention that.

@javiereguiluz
Copy link
Member Author

@wouterj thanks for the review. I made the change you asked for.

@wouterj
Copy link
Member

wouterj commented Sep 18, 2016

👍

@weaverryan
Copy link
Member

Awesome!

@weaverryan weaverryan merged commit abd9a42 into symfony:2.7 Sep 19, 2016
weaverryan added a commit that referenced this pull request Sep 19, 2016
…cle (javiereguiluz) This PR was merged into the 2.7 branch. Discussion ---------- Mentioned the "Symfony Upgrade Fixer" in the upgrade article This fixes #5951. Commits ------- abd9a42 Minor reword 80733b5 Fixed a minor grammar issue 8bdd24b Mentioned the "Symfony Upgrade Fixer" in the upgrade article
@javiereguiluz javiereguiluz deleted the fix_5951 branch May 24, 2018 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

5 participants