Problem/Motivation

#3008870: Drop support for PHPUnit 4.8 once PHP 5 is no longer supported (8.8.x) missed a very small thing for DrupalCI - this doesn't break anything but results in 

00:04:22.641   [InvalidArgumentException]                                            
00:04:22.641   Script "drupal-phpunit-upgrade-check" is not defined in this package

in the logs

Proposed resolution

Remove it. We don't need it.

Remaining tasks

User interface changes

N/a

API changes

N/a

Data model changes

N/a

Release notes snippet

N/a

CommentFileSizeAuthor
#4 3056875-4.patch2.04 KBalexpott
#2 3056875-2.patch1.43 KBalexpott
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexpott created an issue. See original summary.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented
Status: Active » Needs review
FileSize
3056875-2.patch1.43 KB
amateescu’s picture

amateescu CreditAttribution: amateescu for Pfizer, Inc. commented
Status: Needs review » Reviewed & tested by the community

Oops :)

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented
Title: Remove drupal-phpunit-upgrade-check from run-tests.sh » Remove drupal-phpunit-upgrade-check from run-tests.sh and docs relating to drupal-phpunit-upgrade
FileSize
3056875-4.patch2.04 KB
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed 2be8f41 and pushed to 8.8.x. Thanks!

  • catch committed 2be8f41 on 8.8.x 
    Issue #3056875 by alexpott: Remove drupal-phpunit-upgrade-check from run...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.