Skip to content

Fix computing the intersection of Symfony components with extra.symfony.require #580

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
Dec 1, 2019
Merged

Conversation

nicolas-grekas
Copy link
Member

A constraint like ">5.0.0,<5.1" in SYMFONY_REQUIRE doesn't work right now, this fixes it.

@nicolas-grekas
Copy link
Member Author

nicolas-grekas commented Dec 1, 2019

PR ready. FYI the implementation could be a bit more robust if https://flex.symfony.com/versions.json could list the latest stable version of each branch.

e.g.

"branches": {
  "2.7" => "2.7.123",
  etc
}

@nicolas-grekas nicolas-grekas changed the title Fix comparing dev-versions of Symfony components Fix computing the intersection of Symfony components with extra.symfony.require Dec 1, 2019
@fabpot
Copy link
Member

fabpot commented Dec 1, 2019

Thank you @nicolas-grekas.

@fabpot fabpot merged commit 38f2f20 into symfony:master Dec 1, 2019
fabpot added a commit that referenced this pull request Dec 1, 2019
…tra.symfony.require (nicolas-grekas)

This PR was merged into the 1.5-dev branch.

Discussion
----------

Fix computing the intersection of Symfony components with extra.symfony.require

A constraint like `">5.0.0,<5.1"` in `SYMFONY_REQUIRE` doesn't work right now, this fixes it.

Commits
-------

38f2f20 Fix comparing dev-versions of Symfony components
@nicolas-grekas nicolas-grekas deleted the multi branch December 1, 2019 16:30
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.

2 participants