get-binary exits with 0 when the existing binary is kept
維護者通常 1 天內回覆
還沒有人認領這個 Issue。
評估
研究方向
從 GetBinaryCommand::execute() 和 installBinary() 開始，這是 issue 中提到的兩處。execute() 只有在 installBinary() 回傳 null 時才回傳 1，但被略過的檔案也會被回傳。如果 #66 中引用的驗收測試 ExistingBinaryTest::failsWhenBinaryIsKept 存在，請檢查它。當保留既有的二進位檔時以 1 結束並輸出 RoadRunner has not been installed 警告，即為完成。
由索引模型根據 Issue 內容生成。
描述
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.
- 主要語言
- PHP
- 星號
- 30
- 分支
- 15
- 平均合併
- 9 小時 3 分鐘
- 30 天內合併 PR
- 17
環境準備
- 沒有 Dockerfile 或 Docker Compose 檔案
- 沒有 Pull Request 範本
- 閱讀貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
roadrunner-php/cli 的其他 Issue
-
bug
難度 2/5 1-3 小時 新手友好度 72/100
roadrunner-php/cli#67 ·
維護者通常 1 天內回覆
查看 roadrunner-php/cli 的全部 Issue
相似的 Issue
-
難度 2/5 1-3 小時 新手友好度 79/100
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 69/100
bagisto/bagisto-api#54 ·
-
難度 2/5 1-3 小時 新手友好度 64/100
維護者通常 5 天內回覆
-
難度 2/5 1-3 小時 新手友好度 82/100
grokability/snipe-it#19869 · 2 則留言 ·
維護者通常 1 天內回覆
-
regression
難度 2/5 1-3 小時 新手友好度 70/100