Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit ae677cc

Browse files
committed
minor #37823 [Messenger] Don't require doctrine/persistence when installing symfony/messenger (ogizanagi)
This PR was merged into the 5.2-dev branch. Discussion ---------- [Messenger] Don't require doctrine/persistence when installing symfony/messenger | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tickets | Fix #36790 <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead --> | License | MIT | Doc PR | N/A As of today, [installing `symfony/messenger` installs `symfony/doctrine-messenger`](https://github.com/symfony/symfony/blob/3867fb2489ef0d07dec4859cf461bb7bcc7618e0/src/Symfony/Component/Messenger/composer.json#L23) as well. Which means adding `doctrine/persistence` as a hard dep of the latest will install it for any user requiring `symfony/messenger` in 5.2. I think it should stay a soft-dependency until we remove `symfony/doctrine-messenger` as a `symfony/messenger` dependency as of Symfony 6.0 (as mentioned in #35422). --- related: #36785 Commits ------- 5c05455 [Messenger] Don't require doctrine/persistence when installing symfony/messenger
2 parents 4866278 + 5c05455 commit ae677cc

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

src/Symfony/Component/Messenger/Bridge/Doctrine/composer.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
],
1818
"require": {
1919
"php": ">=7.2.5",
20-
"doctrine/persistence": "^1.3",
2120
"symfony/messenger": "^5.1",
2221
"symfony/service-contracts": "^1.1|^2"
2322
},

0 commit comments

Comments
 (0)