Skip to content

#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

Merged
merged 1 commit into from
Jun 1, 2019
Merged

#11627 Minor/Patch version fix #11635

merged 1 commit into from
Jun 1, 2019

Conversation

JoppeDC
Copy link
Contributor

@JoppeDC JoppeDC commented May 31, 2019

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.

@greg0ire
Copy link
Contributor

Indeed:

composer create-project symfony/skeleton my_project && grep symfony my_project/composer.json
        "symfony/console": "4.3.*",
        "symfony/dotenv": "4.3.*",
        "symfony/flex": "^1.1",
        "symfony/framework-bundle": "4.3.*",
        "symfony/yaml": "4.3.*"
        "symfony/polyfill-ctype": "*",
        "symfony/polyfill-iconv": "*",
        "symfony/polyfill-php71": "*",
        "symfony/polyfill-php70": "*",
        "symfony/polyfill-php56": "*"
            "cache:clear": "symfony-cmd",
            "assets:install %PUBLIC_DIR%": "symfony-cmd"
        "symfony/symfony": "*"
        "symfony": {

@greg0ire
Copy link
Contributor

Please squash your commits together.

Copy link

@ghost ghost left a 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.

@weaverryan
Copy link
Member

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

@JoppeDC
Copy link
Contributor Author

JoppeDC commented May 31, 2019

Very good catch! I think there's one more thing that most users will need to update:

symfony/skeleton:composer.json@010e549#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 😉

@JoppeDC
Copy link
Contributor Author

JoppeDC commented Jun 1, 2019

@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.
@wouterj wouterj changed the base branch from master to 4.2 June 1, 2019 12:11
@wouterj wouterj merged commit fc94efc into symfony:4.2 Jun 1, 2019
wouterj added a commit that referenced this pull request Jun 1, 2019
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
wouterj added a commit that referenced this pull request Jun 1, 2019
@wouterj
Copy link
Member

wouterj commented Jun 1, 2019

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!

javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this pull request Jun 3, 2019
* 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
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this pull request Jun 3, 2019
* 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
wouterj added a commit that referenced this pull request Jun 6, 2019
* 4.2:
  Fix several typos
  [#11635] Fixed syntax
  11627 Minor/Patch version fix
  edit normalizer deprecated method on 4.2
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