-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
PHPORM-296 Enable support for Scout v10 #3280
Conversation
@@ -35,12 +35,12 @@ | |||
}, | |||
"require-dev": { | |||
"mongodb/builder": "^0.2", | |||
"laravel/scout": "^11", | |||
"laravel/scout": "^10.3", |
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.
Version 10.3 required for whereNotIn
feature that we implement. laravel/scout@9b45a42
"league/flysystem-gridfs": "^3.28", | ||
"league/flysystem-read-only": "^3.0", | ||
"phpunit/phpunit": "^10.3", | ||
"orchestra/testbench": "^8.0|^9.0", | ||
"mockery/mockery": "^1.4.4", | ||
"mockery/mockery": "^1.4.4@stable", |
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.
This prevents installing unstable release that fails with PHP 8.4
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.
I assume this is needed because of minimum-stability: dev
below. Why does this library have that to begin with?
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.
I think that was to test upcoming versions of Laravel. If found issues before the release thanks to that.
"league/flysystem-gridfs": "^3.28", | ||
"league/flysystem-read-only": "^3.0", | ||
"phpunit/phpunit": "^10.3", | ||
"orchestra/testbench": "^8.0|^9.0", | ||
"mockery/mockery": "^1.4.4", | ||
"mockery/mockery": "^1.4.4@stable", |
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.
I assume this is needed because of minimum-stability: dev
below. Why does this library have that to begin with?
The Scout engine introduced in #3205 works with Scout v10, which is the latest released version. V11 is not released yet.
Fix PHPORM-296