Skip to content

Try to change doctrine/orm version for supporting 2.8 (PHP 8 support). #1155

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

Merged
merged 1 commit into from
Mar 26, 2021

Conversation

GothShoot
Copy link
Contributor

@GothShoot GothShoot commented Mar 25, 2021

Try to use doctrine/orm 2.8 with job queue, this is the last step to php 8 support.

@makasim
Copy link
Member

makasim commented Mar 26, 2021

Have you tested it ?

@GothShoot
Copy link
Contributor Author

Need to setup a work environment because it needs env var to work with your testing config.
I give you a response when I have tested 👍

@makasim
Copy link
Member

makasim commented Mar 26, 2021

You should not require this repo directly. fork read only package, apply your changes and require it in composer.

Once done, backport changes to this repo.

@GothShoot
Copy link
Contributor Author

GothShoot commented Mar 26, 2021

My fork runs on my app (just installing and clear the cache).
Same update as the PR.

@makasim makasim merged commit b32370d into php-enqueue:master Mar 26, 2021
@makasim
Copy link
Member

makasim commented Mar 26, 2021

Good, thanks

@@ -10,7 +10,7 @@
"enqueue/enqueue": "^0.10",
"enqueue/null": "^0.10",
"queue-interop/queue-interop": "^0.8",
"doctrine/orm": "~2.7.4",
"doctrine/orm": "~2.7.4 | ^2.8",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can not this be simplified to "^2.7.4" only?

@makasim
Copy link
Member

makasim commented Apr 29, 2021

fixes #1151

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

Successfully merging this pull request may close these issues.

3 participants