Skip to content

[11.x] Add isTtySupported to Process facade #54604

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Feb 14, 2025
Merged

[11.x] Add isTtySupported to Process facade #54604

merged 3 commits into from
Feb 14, 2025

Conversation

@Riley19280
Copy link
Contributor

@Riley19280 Riley19280 commented Feb 13, 2025

I would like to conditionally turn on TTY for a process if it is supported like so,

Process::when(\Symfony\Component\Process\Process::isTtySupported(), fn (PendingProcess $process) => $process->tty() )

but currently have to reach for the symfony process class to do so. This exposes the symfony Process::isTtySupported to the Process facade.
@Riley19280 Riley19280 changed the title [11.x] Check if TTY is available for a Process [11.x] Add isTtySupported to Process facade Feb 13, 2025
@taylorotwell taylorotwell merged commit 82b0c2f into laravel:11.x Feb 14, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@Riley19280 @taylorotwell