8000 Wire nyholm/psr7 as default factory for PSR-7 by nicolas-grekas · Pull Request #608 · sensiolabs/SensioFrameworkExtraBundle · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Feb 24, 2023. It is now read-only.

Wire nyholm/psr7 as default factory for PSR-7 #608

Merged
merged 1 commit into from
Apr 8, 2019
Merged

Wire nyholm/psr7 as default factory for PSR-7 #608

merged 1 commit into from
Apr 8, 2019

Conversation

nicolas-grekas
Copy link
Collaborator

No description provided.

@nicolas-grekas
Copy link
Collaborator Author

and green

javiereguiluz added a commit to symfony/symfony-docs that referenced this pull request Mar 12, 2019
…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
@fabpot
Copy link
Member
fabpot commented Apr 8, 2019

Thank you @nicolas-grekas.

@fabpot fabpot merged commit 4544cab into sensiolabs:master Apr 8, 2019
fabpot added a commit that referenced this pull request Apr 8, 2019
…rekas)

This PR was merged into the 5.2.x-dev branch.

Discussion
----------

Wire nyholm/psr7 as default factory for PSR-7

Commits
-------

4544cab Wire nyholm/psr7 as default factory for PSR-7
@nicolas-grekas nicolas-grekas deleted the psr7 branch April 8, 2019 14:39
@fabpot fabpot mentioned this pull request Apr 10, 2019
fabpot added a commit that referenced this pull request Apr 10, 2019
This PR was merged into the 5.3.x-dev branch.

Discussion
----------

Fix a regression

introduced in #608, closes #613. The code is from #549 to fix #406

Commits
-------

8120a0a fixed a regression
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0