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

Add other PHP version supports #1

Closed
peter279k opened this issue Oct 7, 2019 · 7 comments · Fixed by #119
Closed

Add other PHP version supports #1

peter279k opened this issue Oct 7, 2019 · 7 comments · Fixed by #119

Comments

@peter279k
Copy link

As title, it seems that this repository misses the php-7.0 and php-7.1 versions.

IMHO, I think we can consider adding these missing PHP version supports.

Once this issue is accepted, I can work on this and please add #orbtoberfest on this issue because of the hacktoberfest circleci :).

@FelicianoTech
Copy link
Contributor

Those versions are too old for this repo. They won't be added.

@the-csaba
Copy link

Hi,

Can you revisit this decision? For example, PHP 5.6 is available in this repo, but 7.0 and 7.1 are not.
I made quick test builds, and everything worked. Would be useful if users will not be forced to use mixed source. Currently, our config look like this:

- image: circleci/php:7.0-cli
- image: circleci/php:7.1-cli
- image: cimg/php:7.2
- image: cimg/php:7.3
- image: cimg/php:7.4
- image: cimg/php:8.0

Thanks

@bednar
Copy link

bednar commented Sep 13, 2021

Hi,

the new “cimg/php” doesn’t supports the PHP 7.1. Is this final solution?

Regards

@FelicianoTech
Copy link
Contributor

Since there is continued requests, I will reopen this Issue. As long as there' no hiccups along the way, this should be resolved soon.

@bednar
Copy link

bednar commented Sep 15, 2021

@FelicianoTech thanks 👍

@FelicianoTech
Copy link
Contributor

Published

@the-csaba
Copy link

Thank you!

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 a pull request may close this issue.

4 participants