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

Use nyholm/psr17 instead of deprecated Zend Diactoros #11135

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@Nyholm
Copy link
Member

Nyholm commented Mar 11, 2019

The DiactorosFactory is deprecated. We should update our docs to use a "pure" PSR-17 factory.

Related to: sensiolabs/SensioFrameworkExtraBundle#608

@javiereguiluz javiereguiluz added this to the 3.4 milestone Mar 12, 2019

javiereguiluz added a commit that referenced this pull request Mar 12, 2019

minor #11135 Use nyholm/psr17 instead of deprecated Zend Diactoros (N…
…yholm)

This PR was submitted for the master branch but it was merged into the 3.4 branch instead (closes #11135).

Discussion
----------

Use nyholm/psr17 instead of deprecated Zend Diactoros

The [DiactorosFactory](https://github.com/symfony/psr-http-message-bridge/blob/master/Factory/DiactorosFactory.php) is deprecated. We should update our docs to use a "pure" PSR-17 factory.

Related to: sensiolabs/SensioFrameworkExtraBundle#608

Commits
-------

dc361b9 Use nyholm/psr17 instead of deprecated Zend Diactoros
@javiereguiluz

This comment has been minimized.

Copy link
Member

javiereguiluz commented Mar 12, 2019

Thanks Tobias! We merged it in 3.4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.