Skip to content

[HttpFoundation] add Early Hints to Reponse to fix test #24764

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
Oct 31, 2017

Conversation

Simperfit
Copy link
Contributor

@Simperfit Simperfit commented Oct 31, 2017

Q A
Branch? 2.7
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets none
License MIT
Doc PR not needed

This line seems to be needed in order to fix the tests, maybe it has been forgot in merge ?

@Simperfit Simperfit force-pushed the hotfix/fix-response-test-for-3.4 branch 2 times, most recently from 9d9d56a to 70b68ae Compare October 31, 2017 06:32
@Simperfit Simperfit changed the title [HttpFoundation] add Early Hints to Reponses to fix tests [HttpFoundation] add Early Hints to Reponse to fix test Oct 31, 2017
@Simperfit Simperfit force-pushed the hotfix/fix-response-test-for-3.4 branch from 70b68ae to c2b4ea2 Compare October 31, 2017 07:07
@Simperfit
Copy link
Contributor Author

Simperfit commented Oct 31, 2017

Travis seems broken.. :(

EDIT: travis back up

@Simperfit Simperfit force-pushed the hotfix/fix-response-test-for-3.4 branch from c2b4ea2 to 0c913fd Compare October 31, 2017 09:14
@chalasr chalasr added this to the 3.4 milestone Oct 31, 2017
@chalasr
Copy link
Member

chalasr commented Oct 31, 2017

The test uses IANA HTTP codes as data provider, and phpdoc says it should be complete.
Should this be considered as a bugfix on 2.7?

@dunglas
Copy link
Member

dunglas commented Oct 31, 2017

Should be merged in 2.7, indeed. 👍

@chalasr chalasr modified the milestones: 3.4, 2.7 Oct 31, 2017
@chalasr chalasr added the Bug label Oct 31, 2017
@Simperfit Simperfit force-pushed the hotfix/fix-response-test-for-3.4 branch from 0c913fd to dbc2f7b Compare October 31, 2017 12:54
@Simperfit Simperfit changed the base branch from 3.4 to 2.7 October 31, 2017 12:54
@Simperfit
Copy link
Contributor Author

PR Rebased

@ogizanagi
Copy link
Contributor

You should re-trigger a travis build by amending your commit.

@Simperfit Simperfit force-pushed the hotfix/fix-response-test-for-3.4 branch from dbc2f7b to 71ad77a Compare October 31, 2017 13:49
@Simperfit
Copy link
Contributor Author

done @ogizanagi

@fabpot
Copy link
Member

fabpot commented Oct 31, 2017

Thank you @Simperfit.

@fabpot fabpot merged commit 71ad77a into symfony:2.7 Oct 31, 2017
fabpot added a commit that referenced this pull request Oct 31, 2017
…imperfit)

This PR was merged into the 2.7 branch.

Discussion
----------

[HttpFoundation] add Early Hints to Reponse to fix test

| Q             | A
| ------------- | ---
| Branch?       | 2.7
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | none
| License       | MIT
| Doc PR        | not needed

This line seems to be needed in order to fix the tests, maybe it has been forgot in merge ?

Commits
-------

71ad77a [HttpFoundation] add Early Hints in Reponse to fix test
@fabpot fabpot mentioned this pull request Nov 5, 2017
@Simperfit Simperfit deleted the hotfix/fix-response-test-for-3.4 branch December 20, 2017 15:41
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.

6 participants