-
Notifications
You must be signed in to change notification settings - Fork 154
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
Fix pacman version with php-fpm #426
Conversation
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove this file
* | ||
* @return string|null | ||
*/ | ||
public function getPhpVersion(): string|null |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Valet should work in PHP <8 (php >7.0) the return type is not required.
Can this fix be kindly merged in ASAP? Unable to install valet on Manjaro/Arch for 2 weeks now due to this php8.2-fpm error. Please advise if there is an alternate workaround. Reverted to using an older version for now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add the using Pacman on line 10:
use Valet\PackageManagers\Pacman;
This PR can not be merged as it is |
Add to PhpFpm->getPhpVersion() new condition to check package manager to skip the version for some bistros, and return the version or null depend on the bistro.
The issue reference to this pull request: Bug: Issue with php8.2-fpm package name. #425