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

Remove php5.3 support #238

Merged
merged 3 commits into from
Jul 15, 2021
Merged

Remove php5.3 support #238

merged 3 commits into from
Jul 15, 2021

Conversation

GRyall
Copy link
Contributor

@GRyall GRyall commented Jul 14, 2020

Depends on ~#190 ~

  • Remove travis tests for php 5.3
  • remove various workarounds to deal with 5.3 not required for 5.4

@GRyall GRyall force-pushed the removePhp5.3 branch 3 times, most recently from 53236a3 to 578f2f1 Compare July 16, 2020 22:40
@GRyall
Copy link
Contributor Author

GRyall commented Jul 20, 2020

I'll move this out of draft once #190 is merged. We won't want to merge it until SL6 is EOL and we no longer support php 5.3

@GRyall GRyall force-pushed the removePhp5.3 branch 2 times, most recently from af30847 to 620898c Compare August 7, 2020 13:01
GRyall added 3 commits August 17, 2020 14:21
php 5.3  is no longer used in production and PR is not compatible with
it.
As part of removing support for PHP 5.3, remove homebrew http response
function as 5.4 features a built in one.
This reverts the patch ont the write API functional tests to  support
php 5.3
@GRyall GRyall marked this pull request as ready for review August 17, 2020 13:22
@GRyall GRyall changed the title Remove php5.3 support - DRAFT Remove php5.3 support Aug 24, 2020
@GRyall GRyall requested a review from gregcorbett October 9, 2020 11:06
@gregcorbett gregcorbett added this to the 5.8.0 milestone Nov 19, 2020
@gregcorbett gregcorbett mentioned this pull request May 20, 2021
@gregcorbett gregcorbett merged commit d95b370 into GOCDB:dev Jul 15, 2021
@tofu-rocketry
Copy link
Member

🥳

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