-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
#11627 Minor/Patch version fix #11635
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
Conversation
Indeed:
|
Please squash your commits together. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me now. Thank you.
Very good catch! I think there's one more thing that most users will need to update: https://github.com/symfony/skeleton/blob/010e5495eb28fa2a5240ef4d75df61c516e36fb3/composer.json#L56 Please test that first - but I'm pretty sure that this is also something that will lock your Symfony deps to, in this example, 4.3 |
Thanks Ryan. I'll check this tomorrow 😉 |
@weaverryan After doing some testing, it appears that you indeed need to set the data setting to the correct version. |
According to the docs the composer.json file is setup for automatic minor version upgrades. This is not the case, its setup for patch version upgrades.
This PR was submitted for the master branch but it was merged into the 4.2 branch instead (closes #11635). Discussion ---------- #11627 Minor/Patch version fix According to the docs the composer.json file is setup for automatic minor version upgrades. This is not the case, its setup for patch version upgrades. <!-- If your pull request fixes a BUG, use the oldest maintained branch that contains the bug (see https://symfony.com/roadmap for the list of maintained branches). If your pull request documents a NEW FEATURE, use the same Symfony branch where the feature was introduced (and `master` for features of unreleased versions). --> Commits ------- fc94efc 11627 Minor/Patch version fix
Thank you @JoppeDC and congratz with your first contribution! I've merged this into 4.2, instead of master, as that's the oldest still maintained branch that this fix should apply (3.4 isn't using Symfony Flex/the skeleton yet). In 2119d98, I've fixed 2 minor syntax errors. Looking forward to see more contributions! |
* 4.2: Remove chrome debug since it's no longer present Fix several typos [symfony#11635] Fixed syntax 11627 Minor/Patch version fix edit normalizer deprecated method on 4.2
* 4.4: Remove chrome debug since it's no longer present Fix several typos [symfony#11635] Fixed syntax 11627 Minor/Patch version fix edit normalizer deprecated method on 4.2
* 4.2: Fix several typos [#11635] Fixed syntax 11627 Minor/Patch version fix edit normalizer deprecated method on 4.2
According to the docs the composer.json file is setup for automatic minor version upgrades. This is not the case, its setup for patch version upgrades.