Skip to content

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 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
May 13, 2019
Merged

Conversation

DavidPrevot
Copy link
Contributor

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.

@nicolas-grekas
Copy link
Member

nicolas-grekas commented May 12, 2019

Thanks, good catch.
For mailer I think it would make sense to list it.
For the bridges I think we prefer keeping them separate.

@nicolas-grekas
Copy link
Member

Thinking twice, bridges should be in the replace section too, so that flex can apply its extra.symfony.require constraints on them!

@DavidPrevot
Copy link
Contributor Author

Updated according to @nicolas-grekas advice, thanks.

@fabpot
Copy link
Member

fabpot commented May 13, 2019

Thank you @DavidPrevot.

@fabpot fabpot merged commit 96f4626 into symfony:4.3 May 13, 2019
fabpot added a commit that referenced this pull request May 13, 2019
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
@fabpot fabpot mentioned this pull request May 22, 2019
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.

4 participants