-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
using different php versions #11
Comments
https://github.com/OskarStark/php-cs-fixer-ga has the same problem/need. you would run the php-cs-fixer in the php version your production machine is running on, because the php language supports different language constructs/keywords. |
Hey Oskar. First let me thank you for your hard work pushing github actions for some often used phptools at this early stage of GA. I was able to learn a lot and do some experiments based on your actions. Finally I came to the conclusion that some tools require to much customizing to make use of your reposes/docker-containers as is.
Instead I am using https://github.com/shivammathur/setup-php For now, which allows some deep customizing of the php environment in GA That way I can even combine several tools in a single action and save some bootstrapping time, see the experiment at staabm/SysMonitor#18 |
I can see this action uses a php 7.3 docker container.
as the results of a phpstan scan depend on the used php version it would be great if this action would support/document how one can run the action using php 7.2/7.1/7.0 etc.
The text was updated successfully, but these errors were encountered: