Skip to content

Refactor Linux and macOS tool checks for improved error handling and command execution#1025

Merged
crazywhalecc merged 1 commit intov3-devfrom
v3-fix/doctor-command
Feb 6, 2026
Merged

Refactor Linux and macOS tool checks for improved error handling and command execution#1025
crazywhalecc merged 1 commit intov3-devfrom
v3-fix/doctor-command

Conversation

@crazywhalecc
Copy link
Owner

@crazywhalecc crazywhalecc commented Feb 6, 2026

What does this PR do?

These are the only sudo commands that SPC used. SUDO needs password input, we cannot use proc_open to handle it for now.

Checklist before merging

If your PR involves the changes mentioned below and completed the action, please tick the corresponding option.
If a modification is not involved, please skip it directly.

  • If you modified *.php or *.json, run them locally to ensure your changes are valid:
    • composer cs-fix
    • composer analyse
    • composer test
    • bin/spc dev:sort-config
  • If it's an extension or dependency update, please ensure the following:
    • Add your test combination to src/globals/test-extensions.php.
    • If adding new or fixing bugs, add commit message containing extension test or test extensions to trigger full test suite.

@crazywhalecc crazywhalecc merged commit 82bf317 into v3-dev Feb 6, 2026
1 check passed
@crazywhalecc crazywhalecc deleted the v3-fix/doctor-command branch February 6, 2026 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant