get-binary exits with 0 when the existing binary is kept
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue nà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
- 82/100
Hướng nghiên cứu
Bắt đầu từ GetBinaryCommand::execute() và installBinary(), hai phần mà issue nêu tên: execute() chỉ trả về 1 khi installBinary() trả về null, nhưng tệp bị bỏ qua vẫn được trả về. Kiểm tra bài kiểm thử chấp nhận ExistingBinaryTest::failsWhenBinaryIsKept được nhắc đến trong #66 nếu nó có tồn tại. Hoàn thành khi việc giữ lại một binary đã có kết thúc với mã 1 và in ra cảnh báo RoadRunner has not been installed.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Description
When the binary already exists and is not overwritten — the overwrite question is answered "no", or the command runs with --no-interaction — rr get keeps the old file (correct) but exits with 0 and prints [OK] Your project is now ready. Scripts cannot tell that nothing was installed.
GetBinaryCommand::execute() returns 1 with RoadRunner has not been installed only when installBinary() returns null, but installBinary() returns the skipped file as well.
Expected: exit code 1 and the RoadRunner has not been installed warning when the existing binary is kept.
Steps to reproduce
- Put any file named
rrinto an empty directory. - Run
rr get --no-config -nthere. - The file is unchanged, the exit code is 0.
Additional context
Found by the acceptance tests in #66 (ExistingBinaryTest::failsWhenBinaryIsKept). The DLoad-based implementation in #49 already returns 1 in this case.
- Ngôn ngữ chính
- PHP
- Star
- 30
- Fork
- 15
- Merge trung bình
- 45 phút
- Pull request đã merge (30 ngày)
- 11
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
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 roadrunner-php/cli
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
roadrunner-php/cli#68 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
--stability=rc is treated as devĐang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
roadrunner-php/cli#67 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
roadrunner-php/cli#62 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
roadrunner-php/cli#61 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
roadrunner-php/cli#60 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của roadrunner-php/cli
Issue tương tự
-
sync-en
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Maintainer thường phản hồi trong vòng 1 ngày
-
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 4 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Refund Analytics Tool — “Check” Button Fails to Render in WooCommerce 11.2.1Có thể đã có người làm @MinhazI đã nhận hôm nay. Đang mởBug Reports/Analytics type: community contribution
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
woocommerce/woocommerce#69689 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 60/100
Maintainer thường phản hồi trong vòng 2 ngày