Problem/Motivation

In #3362083: '@requires externalCommand' is not parsed in PHPUnit 10 we removed @requires externalCommand but BuildTestBase still mentions it:

 * - Allow the use of PHPUnit-style (at)require annotations for external shell
 *   commands.

Steps to reproduce

Proposed resolution

Remove the comment above.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3377665

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

longwave created an issue. See original summary.

sourabhjain made their first commit to this issue’s fork.

sourabhjain’s picture

sourabhjain
Primary language English
commented
Assigned: Unassigned » sourabhjain
Status: Active » Needs work

sourabhjain opened merge request !4499

sourabhjain’s picture

sourabhjain
Primary language English
commented
Assigned: sourabhjain » Unassigned
Status: Needs work » Needs review

I have done with the proposed solution. Please review MR.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Reviewed & tested by the community

Thanks!

  • quietone committed f42411a0 on 11.x 
    Issue #3377665 by sourabhjain, longwave: Remove reference to @requires...
quietone’s picture

quietone commented
Status: Reviewed & tested by the community » Fixed

I read the related issue to learn more about this removal, thanks for that. And I do like a bit of tidying up :-)

Committed f42411a and pushed to 11.x. Thanks!

quietone closed merge request !4499

Status: Fixed » Closed (fixed)

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