Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
PHP 7 complete's test runs 1/3rd faster than PHP 5.6. This saves overall resources for the testing infrastructure, as well as giving quicker feedback to patch authors.
https://dispatcher.drupalci.org/job/php7_mysql5.5/133/ 21 minutes.
https://dispatcher.drupalci.org/job/php5.6_mysql5.5/173/ 30 minutes.
Also in general I think we should emphasize support for the most recent stable PHP version
Proposed resolution
Default patch testing to PHP 7, switch PHP 5.3 to per-commit testing.
Do this as soon as PHP 7.0 is out or when the two testing hacks in #2454439: [META] Support PHP 7 have been reverted. PHP 7.0 is already out, D8 has already switched to PHP 7.0 and D7 should as well.
Remaining tasks
Decide that we're going to do it, then postpone on #2454439: [META] Support PHP 7.
postpone on #2656548: Fully support PHP 7.0 in Drupal 7
Then Do it.
User interface changes
none
API changes
none
Data model changes
none
Comments
Comment #2
joseph.olstadComment #3
joseph.olstadComment #4
joseph.olstadI've already done this for these projects, or plan to do so shortly.
media,
i18n
file_entity
media_ckeditor
multiform
sassy
prepro
Comment #5
joseph.olstadComment #6
joseph.olstadPostpone on
#2656548: Fully support PHP 7.0 in Drupal 7
Comment #7
David_Rothstein CreditAttribution: David_Rothstein as a volunteer commentedI went ahead and made this change now, as discussed previously at #2607222: [policy, no patch] Default to PHP 7 for Drupal core patch testing. I didn't see any reason to postpone it - PHP 7 has been fully supported (with all tests passing) for a long time. But if something goes wrong we can easily revert it.
Comment #8
joseph.olstadThanks @DavidRothstein !