diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-08-18 08:09:57 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-08-18 08:09:57 +0900 |
| commit | b94e722ab71e06ab1f8a0be0ac4b76ba280fc73b (patch) | |
| tree | c33982caee404ae9ca52aa4459c53756e12bd2ff /crates/shirabe-symfony-console/src/command/list_command.rs | |
| parent | da59204f1df8ee3e27ee4385ae81512a188bf567 (diff) | |
| download | php-shirabe-b94e722ab71e06ab1f8a0be0ac4b76ba280fc73b.tar.gz php-shirabe-b94e722ab71e06ab1f8a0be0ac4b76ba280fc73b.tar.zst php-shirabe-b94e722ab71e06ab1f8a0be0ac4b76ba280fc73b.zip | |
fix(diagnose-command): model check results as dedicated enums
The `check_*` methods returned `PhpMixed`, and `output_result` reproduced
PHP's truthiness over it incorrectly: only `PhpMixed::Bool(false)` took the
falsey path, so an empty string or an empty list printed a blank line
instead of reporting `FAIL` and raising the exit code. `CheckResult` and
`GithubRateLimit` make the shapes PHP returns explicit, and `check_platform`
stores the detail of an error or warning as `Option<String>` rather than
`true`-or-string.
`check_http` and `check_composer_repo` also built the exception label with
`std::any::type_name_of_val`, printing the Rust type path where PHP prints
`get_class($e)`; they now go through `AnyThrowable::php_class_name` like the
rest of the file.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe-symfony-console/src/command/list_command.rs')
0 files changed, 0 insertions, 0 deletions
