Skip to content
This repository has been archived by the owner on Feb 13, 2023. It is now read-only.

Include php_xdebug_version in default.config.yml #1997

Merged
merged 1 commit into from
Apr 26, 2020

Conversation

joel-travieso
Copy link
Contributor

Include php_xdebug_version in default.config.yml to provide visibility to this option.

Include php_xdebug_version in default.config.yml to provide visibility to this option
@geerlingguy
Copy link
Owner

Just wondering, what's the impetus here (why did you need to change the version from the default)?

@joel-travieso
Copy link
Contributor Author

Just wondering, what's the impetus here (why did you need to change the version from the default)?

@geerlingguy I found compatibility problems between the default version (2.6.0) and PHP 7.3. I struggled a bit to find a way to change that, so I thought adding it here would save others the time.

@geerlingguy
Copy link
Owner

I'll merge this in but I'll also set the default in the playbook since the version of PHP Drupal VM defaults to may shift from what the upstream role defaults to over time, and it's good to test the versions in lockstep.

Note that the current upstream default is 2.9.0...

@geerlingguy geerlingguy merged commit ec956e4 into geerlingguy:master Apr 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants