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

Add conflict with doctrine/mongodb-odm <2.0 #240

Merged
merged 2 commits into from
Sep 23, 2020

Conversation

franmomu
Copy link
Member

@franmomu franmomu commented Sep 22, 2020

Subject

Ref: sonata-project/dev-kit#1069

We are dropping support for doctrine/mongodb-odm <2.0, this allows us to not install alcaeus/mongo-php-adapter in GA for tests.

I am targeting this branch, because these changes are BC.

Changelog

### Removed
- Remove support for `doctrine/mongodb-odm` <2.0

We are dropping support for doctrine/mongodb-odm 2.0, this
allows us to not install alcaeus/mongo-php-adapter in GA for tests
jordisala1991
jordisala1991 previously approved these changes Sep 22, 2020
phansys
phansys previously approved these changes Sep 22, 2020
Copy link
Member

@phansys phansys left a comment

Choose a reason for hiding this comment

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

I think the PR description is missing the < operator in the version constraint:

We are dropping support for doctrine/mongodb-odm 2.0

core23
core23 previously approved these changes Sep 22, 2020
@@ -25,10 +25,13 @@
"doctrine/dbal": "^2.6",
"doctrine/persistence": "^1.3.6 || ^2.0"
},
"conflict": {
"doctrine/mongodb-odm": "<2.0"
},
"require-dev": {
"doctrine/common": "^2.7 || ^3.0",
"doctrine/doctrine-bundle": "^2.0",
Copy link
Member

Choose a reason for hiding this comment

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

Can you please add a conflict entry for the other doctrine deps too?

They are required as dev dependencies.
This gives more control over which versions are installed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants