The PHP version check in prepare.php still allows PHP 7.2 and 7.3
Maintainer thường phản hồi trong vòng 2 ngày
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 72/100
Hướng nghiên cứu
Bắt đầu tại prepare.php, ở phần kiểm tra phiên bản PHP được mô tả trong issue, sau đó đọc $required_php_version trong src/wp-includes/version.php của checkout đã chuẩn bị. Tái hiện trường hợp được báo cáo bằng PHP 7.3.33 và WPT_PHP_EXECUTABLE; hoàn thành khi bước kiểm tra từ chối các phiên bản thấp hơn mức tối thiểu của WordPress bằng một thông báo rõ ràng trước khi bắt đầu cài đặt các dependency.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- PHP
- Star
- 77
- Fork
- 79
- Merge trung bình
- 4 ngày 19 giờ
- Pull request đã merge (30 ngày)
- 3
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của WordPress/phpunit-test-runner
-
make testrunner.sh in host-tools directoryCó thể đã có người làm @iarif4u đã nhận 19 ngày trước. Đang mởContributorDay tooling WCUS26
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
WordPress/phpunit-test-runner#260 · 4 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
create host-tools directory in root folderCó thể đã có người làm @iarif4u đã nhận 19 ngày trước. Đang mởContributorDay tooling WCUS26
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 78/100
WordPress/phpunit-test-runner#259 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
A database password with a quote or a backslash breaks wp-tests-config.phpCó thể đã có người làm @ekamran đã nhận 9 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 25/100
WordPress/phpunit-test-runner#354 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Error: Error uploading results: Invalid parameter(s): message (HTTP status 400)`Có thể đã có người làm @ekamran đã nhận 21 ngày trước. Đang mởbug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
WordPress/phpunit-test-runner#335 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 2 ngày
-
enhancement tooling WCUS26
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
WordPress/phpunit-test-runner#316 ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của WordPress/phpunit-test-runner
Issue tương tự
-
needs-triage Platform(Default)
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 2 ngày
-
sync-en
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[Sync EN] Document Error thrown by shm_* functions on a detached segment (#5975)Có thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mởsync-en
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 3 ngày
-
Перевод устарел
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100