The PHP version check in prepare.php still allows PHP 7.2 and 7.3
Los mantenedores suelen responder en 2 días
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 72/100
Línea de trabajo
Empieza en prepare.php, en la comprobación de la versión de PHP descrita en el issue, y después lee $required_php_version en src/wp-includes/version.php del checkout preparado. Reproduce el caso descrito con PHP 7.3.33 y WPT_PHP_EXECUTABLE; se considera terminado cuando la comprobación rechaza las versiones inferiores al mínimo de WordPress con un mensaje claro antes de que comience la instalación de dependencias.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
prepare.php stops a run when the test environment's PHP is older than 7.2:
if ( version_compare( $env_php_version, '7.2', '<' ) ) {
7.2 was the WordPress minimum when the check was last raised in 33584bb. WordPress 7.0 raised the minimum to PHP 7.4 (r61459, #62622), and the README here already lists PHP 7.4+, but the check still lets 7.2 and 7.3 through.
I ran prepare.php from master (31185de) against current trunk with WPT_PHP_EXECUTABLE pointing to a script that reports PHP 7.3.33. The check passed, the run cloned wordpress-develop, ran npm install and the build, and then stopped in Composer:
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Root composer.json requires php >=7.4 but your php version (7.3.33; overridden via config.platform, actual: 8.5.11) does not satisfy that requirement.
So a host on PHP 7.2 or 7.3 gets a dependency error after the build, instead of a message that its PHP version is too old.
WordPress keeps its minimum in $required_php_version in src/wp-includes/version.php, which is already in the prepared checkout. Reading it from there would keep the check in line with WordPress without editing it again. I have a fix and will open a pull request.
- Lenguaje dominante
- PHP
- Estrellas
- 77
- Forks
- 79
- Merge medio
- 4 d 19 h
- PR fusionados (30 d)
- 3
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de WordPress/phpunit-test-runner
-
make testrunner.sh in host-tools directoryPosiblemente ocupada @iarif4u la tomó hace 17 días. AbiertoContributorDay tooling WCUS26
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
WordPress/phpunit-test-runner#260 · 4 comentarios ·
Los mantenedores suelen responder en 2 días
-
create host-tools directory in root folderPosiblemente ocupada @iarif4u la tomó hace 17 días. AbiertoContributorDay tooling WCUS26
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
WordPress/phpunit-test-runner#259 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
A database password with a quote or a backslash breaks wp-tests-config.phpPosiblemente ocupada @ekamran la tomó hace 7 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 25/100
WordPress/phpunit-test-runner#354 ·
Los mantenedores suelen responder en 2 días
-
Error: Error uploading results: Invalid parameter(s): message (HTTP status 400)`Posiblemente ocupada @ekamran la tomó hace 19 días. Abiertobug
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
WordPress/phpunit-test-runner#335 · 1 asignado ·
Los mantenedores suelen responder en 2 días
-
enhancement tooling WCUS26
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
WordPress/phpunit-test-runner#316 ·
Los mantenedores suelen responder en 2 días
Todos los issues de WordPress/phpunit-test-runner
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Automattic/blocks-engine#3090 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
macOS
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
beyondcode/herd-community#1771 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
-
module: user interface priority: high severity: minor type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día