Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
Update dependencies in the main component#31480
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
nicolas-grekas commentedMay 12, 2019 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Thanks, good catch. |
nicolas-grekas commentedMay 12, 2019
Thinking twice, bridges should be in the replace section too, so that flex can apply its extra.symfony.require constraints on them! |
DavidPrevot commentedMay 13, 2019
Updated according to@nicolas-grekas advice, thanks. |
fabpot commentedMay 13, 2019
Thank you@DavidPrevot. |
This PR was squashed before being merged into the 4.3 branch (closes#31480).Discussion----------Update dependencies in the main component| Q | A| ------------- | ---| Branch? | 4.3| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | n/a| License | MIT| Doc PR |The doctrine/collections and psr/simple-cache dependencies have already been demoted to require-dev in the relevant components.BTW, I wonder if symfony/mailer (and symfony/*-mailer bridges) should be provided in the main composer.json too.Commits-------96f4626 Update dependencies in the main component
The doctrine/collections and psr/simple-cache dependencies have already been demoted to require-dev in the relevant components.
BTW, I wonder if symfony/mailer (and symfony/*-mailer bridges) should be provided in the main composer.json too.