Skip to content
New issue

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

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

Already on GitHub? # to your account

Update symfony/process #52

Merged
merged 1 commit into from
Mar 23, 2024
Merged

Conversation

CalumChamberlain
Copy link
Contributor

Description

Updated the symfony/process dependancy to allow version 7

Motivation and context

Why is this change required? What problem does it solve?

Laravel 11 recently released and now required symfony/process: 7

How has this been tested?

Please describe in detail how you tested your changes.

I forked this project, updated the dependancy and required into our upgraded Laravel 11 project and all seems to be working as before.

Checklist:

Go over all the following points before making your PR:

  • I have read the CONTRIBUTING document.
  • My pull request addresses exactly one patch/feature.
  • I have created a branch for this patch/feature.
  • I have added tests to cover my changes.
  • If my change requires a change to the documentation, I have updated it accordingly.

If you're unsure about any of these, don't hesitate to ask. We're here to help!

@tigitz tigitz merged commit ea379a8 into tigitz:master Mar 23, 2024
@tigitz
Copy link
Owner

tigitz commented Mar 23, 2024

Thank you @CalumChamberlain

@jwhulette
Copy link

@tigitz Could we get a new tag with this included for Laravel 11 support?

@tigitz
Copy link
Owner

tigitz commented May 4, 2024

@jwhulette, thanks for the reminder!

0.7.0 is now released

I've also taken the time to update the library to fully support PHP 8.3.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants