search

lando / drupal

The Official Drupal Lando plugin
https://docs.lando.dev/drupal
GNU General Public License v3.0
15 stars 16 forks source link

PHP and Drush versions are not configurable #49

Closed agarzola closed 1 year ago

agarzola commented 1 year ago

The PHP version and Drush version are touted as configurable in the Drupal plugin doc site:

  • Configurable php version from 5.3 all the way to 8.1
  • Configurable composer and drush

However, that is not consistent with the code, as values are hardcoded for php and drush in the drupal9 recipe builder (and other recipes appear to also hardcode these values): https://github.com/lando/drupal/blob/155d20373fdc6544254f7c27ea40eafb2d4117b6/recipes/drupal9/builder.js#L14-L21

Indeed, the following .lando.yml config still installs PHP 8.0:

name: test-project
recipe: drupal9

config:
  php: 8.1

It would be very helpful to make these configurable, as the documentation suggests.

agarzola commented 1 year ago

False alarm! This is actually working as intended. I was able to configure the PHP version in another project, so something must be interfering with my configuration somehow.