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

Mentioned the "Symfony Upgrade Fixer" in the upgrade article #6941

Merged
merged 3 commits into from
Sep 19, 2016

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants