-
Notifications
You must be signed in to change notification settings - Fork 200
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
PHP 8 support #344
Comments
@saimaz Could you please bump this package? Don't why the open PR has changes, does this really rewriting for PHP 8? |
Hello, Any news on this issue? Thanks! |
As a temporary workaround, I published https://packagist.org/packages/ocramius/elasticsearch-dsl#7.9.999 This allows usage within my local infrastructure, but it comes with ZERO BC promise, since I'd rather see it deleted when @ongr-io releases a compatible version :-) |
Or just use https://github.com/handcraftedinthealps/elasticsearchdsl. It‘s maintained by the guys of sulu so it‘s in good hands i guess. Also supports symfony5 and php8. |
That's something I've already done, but we should support @ongr-io first. |
hey, i'm currently testing the php8 integration with a local project, will try to release a version with support soonish. Changes are on https://github.com/ongr-io/ElasticsearchDSL/tree/7.2 already i just need to test it and the release the next version (maybe i'll copy a release pipeline from one of @Ocramius projects for that - but i don't have that much experience with github actions yet) |
@84m feel free to poke/call me if you need any guidance: anything that brings this lib forward is immediate value :-) Alternatively, check https://github.com/laminas/laminas-continuous-integration-action, which does a bunch of checks, all in one shot. |
It will be great to add php8 support(as dependency)
https://www.php.net/releases/8.0/en.php
The text was updated successfully, but these errors were encountered: