We currently set the default minimum PHP version at 7.4, which is much older than our policy requires*:
|
<!-- PHP Compatibility sniffs --> |
|
<!-- The lowest version of PHP supported by both Drupal and Acquia Cloud. |
|
@see https://www.drupal.org/docs/8/system-requirements/php-requirements |
|
@see https://docs.acquia.com/acquia-cloud/arch/tech-platform/ |
|
For instructions on overriding this value in your project, |
|
@see https://github.com/acquia/coding-standards-php/pull/37 --> |
|
<config name="testVersion" value="7.4-"/> |
|
<rule ref="PHPCompatibility"> |
|
<exclude name="PHPCompatibility.Extensions.RemovedExtensions.famRemoved"/> |
|
</rule> |
* Acquia technically has a PHP 7.4 LTS subscription extension, but I think we should consider that an edge case with an easy workaround.
We should discuss whether we want to update it to 8.1 (Drupal core's current minimum supported version) and cut a new release or take a whole different approach--like moving the whole package phpcompatibility/php-compatibility to suggest since setting it directly has known complications around it: #85.
What do you think, @danepowell?
We currently set the default minimum PHP version at 7.4, which is much older than our policy requires*:
coding-standards-php/src/Standards/AcquiaPHPMinimal/ruleset.xml
Lines 19 to 28 in 7eaa70c
* Acquia technically has a PHP 7.4 LTS subscription extension, but I think we should consider that an edge case with an easy workaround.
We should discuss whether we want to update it to 8.1 (Drupal core's current minimum supported version) and cut a new release or take a whole different approach--like moving
the whole packagephpcompatibility/php-compatibilitytosuggestsince setting it directly has known complications around it: #85.What do you think, @danepowell?