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

[DoctrineBridge] Bump doctrine/data-fixtures #37640

Merged
merged 1 commit into from Jul 23, 2020

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Jul 23, 2020

Q A
Branch? 3.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets N/A
License MIT
Doc PR N/A

Discovered while working on #37639: DoctrineBridge is locked to an old an unmaintained branch of doctrine/data-fixtures. This is going to be a problem as soon as we want to support doctrine/persistence 2.

@nicolas-grekas nicolas-grekas added this to the 3.4 milestone Jul 23, 2020
@nicolas-grekas
Copy link
Member

Thank you @derrabus.

@nicolas-grekas nicolas-grekas merged commit 700f68f into symfony:3.4 Jul 23, 2020
nicolas-grekas added a commit that referenced this pull request Jul 23, 2020
This PR was squashed before being merged into the 4.4 branch.

Discussion
----------

Allow doctrine/persistence 2

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | N/A
| License       | MIT
| Doc PR        | N/A

Replaces #37416.

This PR unblocks the installation of `doctrine/persistence` version 2.

Notes:
* I had to bump `doctrine/data-fixtures` because composer would otherwise downgrade to an ancient alpha release with incomplete version constraints. This package is a dev dependency for our tests, so I think we're good here.
  **edit**: Moved to #37640.
* Until doctrine/data-fixtures#349 has been resolved, Doctrine/Bridge cannot be tested with version 2 of `doctrine/persistence`.

Commits
-------

cd22fe6 Allow doctrine/persistence 2
@derrabus derrabus deleted the bugfix/data-fixtures branch July 23, 2020 21:11
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.

None yet

5 participants