-
Notifications
You must be signed in to change notification settings - Fork 12
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
OEL-1322: PHP 8.0 Upgrade. #209
Conversation
c8cc2bc
to
e5e558f
Compare
.drone.yml
Outdated
@@ -17,6 +17,7 @@ services: | |||
image: registry.fpfis.eu/fpfis/httpd-php:${PHP_VERSION}-ci | |||
environment: | |||
- DOCUMENT_ROOT=/test/oe_bootstrap_theme | |||
- COMPOSERVER=--2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We removed this already recently, no need to readd.
Seems good to me. For the changes in composer.json, we need to make sure to clean up if we no longer need these packages. E.g. token is only needed for ui_patterns_settings, and perhaps we will remove this in the future? I also don't know if there are any yet undiscovered problems for PHP 8.0. |
Btw the README.md still talks about PHP 7.3 as a minimum version. |
Jira issue(s):
https://citnet.tech.ec.europa.eu/CITnet/jira/browse/OEL-1322