Skip to content

Stop support for PHP 5.5 #37

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 2 commits into from
Jul 30, 2019
Merged

Conversation

nenad
Copy link
Contributor

@nenad nenad commented Jul 29, 2019

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? yes
Related tickets no
Documentation no
License MIT

What's in this PR?

Stop support for PHP 5.5

Why?

Prevents us from migrating to newer version of react components.

@nenad nenad mentioned this pull request Jul 29, 2019
3 tasks
Copy link
Contributor

@dbu dbu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think we should bump all the way to PHP 7.1 while we are at it. and increase the minor version in the composer branch-alias so we will be 2.2. i guess in this case its very unlikely to matter, but just out of principle so that we can release security fixes that people stuck on legacy php versions.

Copy link
Member

@xabbuh xabbuh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In fact, we need to update the PHP version for the job that runs with lowest dependencies too.

@nenad nenad force-pushed the patch/stop-support-5-5 branch from 4c8e008 to dbd3f10 Compare July 30, 2019 08:43
@nenad
Copy link
Contributor Author

nenad commented Jul 30, 2019

In fact, we need to update the PHP version for the job that runs with lowest dependencies too.

I removed PHP 5.5 from the Travis build completely. Does this comment refer to adding the lowest dependencies job to PHP 7.1 or some other version?

@dbu dbu merged commit 5d75e5b into php-http:master Jul 30, 2019
@nenad nenad deleted the patch/stop-support-5-5 branch July 30, 2019 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants