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

Update Drupal 9 recipe to PHP 8.0 #84

Merged
merged 3 commits into from
Dec 17, 2021
Merged

Update Drupal 9 recipe to PHP 8.0 #84

merged 3 commits into from
Dec 17, 2021

Conversation

cabagach
Copy link
Contributor

Recommendation
Use PHP version 8.0 or higher for current release versions of Drupal 9. Drupal 10 is planned to require PHP 8.

From the Drupal 9 PHP requirements.

> Use PHP version 8.0 or higher for current release versions of Drupal 9. Drupal 10 is planned to require PHP 8.

From Drupal 9 [PHP requirements](https://www.drupal.org/docs/system-requirements/php-requirements).
@labboy0276
Copy link
Sponsor Contributor

labboy0276 commented Dec 17, 2021

I would do 8.0 for now as 8.1 is fairly new and there are still some extensions that don't work with it. @cabagach

@cabagach
Copy link
Contributor Author

Yeah, probably best to be safe, I have updated the version to 8.0.

@cabagach cabagach changed the title Update Drupal 9 recipe to PHP 8.1 Update Drupal 9 recipe to PHP 8.0 Dec 17, 2021
@labboy0276
Copy link
Sponsor Contributor

@cabagach looks like you need to update the tests too since it is looking for php 7.4: https://github.com/lando/cli/tree/main/examples/drupal9

@labboy0276 labboy0276 merged commit 317e054 into lando:main Dec 17, 2021
@cabagach
Copy link
Contributor Author

Thanks for the fast response @labboy0276, and also for helping me make the test pass.

@labboy0276
Copy link
Sponsor Contributor

No problem, it will be in the next release of Lando @cabagach

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.

None yet

2 participants