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

Deprecate PHP 7.1, 7.2, support 7.4 #526

Closed
5 tasks done
bagofarms opened this issue May 15, 2020 · 5 comments · Fixed by #578
Closed
5 tasks done

Deprecate PHP 7.1, 7.2, support 7.4 #526

bagofarms opened this issue May 15, 2020 · 5 comments · Fixed by #578

Comments

@bagofarms
Copy link
Member

bagofarms commented May 15, 2020

Now that we have support for PHP 7.3, and 7.1 and 7.2 are no longer actively updated (7.2 gets security fixes only through November), we need to:

  • Remove PHP 7.1 and 7.2 support from documentation
  • Remove Travis tests for anything older than 7.3
  • Add Travis tests for 7.3 and 7.4
  • Make any code updates required
  • Update Composer and packages as required
@lukeislucas
Copy link

Is there any timeline on this item? We are trying to install this on a PHP 7.4 server and ran into issues.

@bagofarms
Copy link
Member Author

I expect to start working on this over the next couple of weeks, but I don't know what kinds of issues we'll run into. Could you provide some more information on the issues you encountered?

@lukeislucas
Copy link

When running the install via php composer.phar install, the following error stops the installation:

Your requirements could not be resolved to an installable set of packages.
Problem 1
- This package requires php ~7.2.0 || ~7.3.0 but your PHP version (7.4.6) does not satisfy that requirement.

We are in the process of migrating a working version from PHP 7.2 over to our PHP 7.4 server, but we have not had a chance to test success in that scenario yet.

@bagofarms bagofarms modified the milestones: v2.8.0, v2.7.1 Dec 4, 2020
@bagofarms
Copy link
Member Author

Related to #546

@bagofarms
Copy link
Member Author

bagofarms commented May 24, 2021

Some of this was done in #559

@AlanFCMV AlanFCMV linked a pull request May 27, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants