Skip to content
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.

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

Use overriden storm process in console commands #1117

Closed
wants to merge 1 commit into from

Conversation

mjauvin
Copy link
Member

@mjauvin mjauvin commented May 3, 2024

Depends on wintercms/storm#175

Use our overriden Process class that will warn users when open_basedir restrictions are in effect, and let the process be created without TTY mode enabled.

@mjauvin mjauvin added maintenance PRs that fix bugs, are translation changes or make only minor changes needs review Issues/PRs that require a review from a maintainer labels May 3, 2024
@mjauvin mjauvin added this to the 1.2.7 milestone May 3, 2024
@mjauvin mjauvin self-assigned this May 3, 2024
@mjauvin mjauvin requested a review from LukeTowers May 4, 2024 02:42
@mjauvin mjauvin added blocked Issues/PRs that cannot proceed at this point in time and removed needs review Issues/PRs that require a review from a maintainer labels May 4, 2024
@mjauvin mjauvin added stale Issues/PRs that have had no activity and may be archived and removed blocked Issues/PRs that cannot proceed at this point in time maintenance PRs that fix bugs, are translation changes or make only minor changes labels May 17, 2024
@mjauvin
Copy link
Member Author

mjauvin commented May 17, 2024

Fixed upstream in Symfony/Process

@mjauvin mjauvin closed this May 17, 2024
@mjauvin mjauvin removed this from the 1.2.7 milestone May 17, 2024
@LukeTowers LukeTowers deleted the use-storm-process branch May 17, 2024 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Issues/PRs that have had no activity and may be archived
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant