Problem/Motivation

Child issue of #3537713: [12.x] Require PHP 8.5
See PHPStan report https://git.drupalcode.org/issue/drupal-3537713/-/jobs/7716644#L680

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#14 3564075-nr-bot_i16xe3am.txt91 bytesneeds-review-queue-bot

Issue fork drupal-3564075

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

voleger created an issue. See original summary.

mondrake’s picture

mondrake
Location 🇮🇹
commented

That code is removed in #3515347: Reduce run-tests.sh complexity in spawning subprocesses.

mondrake’s picture

mondrake
Location 🇮🇹
commented
Related issues: +#3515347: Reduce run-tests.sh complexity in spawning subprocesses
voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented
Status: Active » Closed (won't fix)

Indeed, let's link those issues.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

mondrake’s picture

mondrake
Location 🇮🇹
commented
Status: Closed (won't fix) » Active

Well, related issue is still NR, so if it won't make it soon, we'll have the problem anyway. Let's keep this open IMO.

voleger opened merge request !14149

voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented
Status: Active » Needs review
mondrake’s picture

mondrake
Location 🇮🇹
commented
Issue tags: +run-tests.sh
quietone’s picture

quietone commented
Version: 11.3.x-dev » 11.x-dev

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Based on the other one, lets just go ahead and land this one.

quietone’s picture

quietone commented
Title: Backtick operator is deprecated in PHP 8.5 in core/scripts/run-tests.sh » Remove use of backtick operator in run-tests.sh for PHP 8.5
needs-review-queue-bot’s picture

needs-review-queue-bot commented
Status: Reviewed & tested by the community » Needs work
StatusFileSize
new 3564075-nr-bot_i16xe3am.txt91 bytes

The Needs Review Queue Bot tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs work » Closed (outdated)

#3515347: Reduce run-tests.sh complexity in spawning subprocesses landed and the backticks were removed there.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

voleger closed merge request !14149