Skip to content
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

[wip] Support for ORM 3 #753

Open
wants to merge 6 commits into
base: 6.2.x
Choose a base branch
from

Conversation

TomHAnderson
Copy link
Member

Composer resolves to highest versions (except phpunit)

@TomHAnderson TomHAnderson added this to the 7.0.0 milestone Mar 1, 2024
@TomHAnderson TomHAnderson self-assigned this Mar 2, 2024
/** @ORM\Column(type="bigint") */
protected int $bigint;
#[ORM\Column(type: 'bigint', nullable: false)]
protected string $bigint;
Copy link
Member Author

Choose a reason for hiding this comment

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

old code was incorrectly cast to int

@demiankatz
Copy link
Contributor

demiankatz commented Jan 24, 2025

@TomHAnderson, I see that this hasn't moved in a while. I'm likely to need it sooner or later. Is there anything I can do to help things along?

[edit: I've followed the trail a little further, and I see that you're running into some of the same interlocking Laminas dependency changes that have been causing trouble on my end as well... we may just have to keep waiting for some things, but if you need Laminas-related help to make progress, feel free to call on me and I'll do what I can as time permits.]

@driehle
Copy link
Member

driehle commented Jan 25, 2025

@demiankatz We cannot provide support for ORM 3 in a new minor release. The reason is stated in doctrine/DoctrineModule#833. We need to switch DoctrineModule from doctrine/cache to psr/cache, which results in a BC break. I have, however, prioritized support for PHP 8.4, which will be available with 6.3.0, as soon as laminas/laminas-serializer#73 is merged and released.

@demiankatz
Copy link
Contributor

Thanks, @driehle, I definitely understand that a major release will be needed for ORM 3. I appreciate the incremental progress on minor releases in the meantime, but I'm also willing to help out as needed to get to the next major version (though I also realize that there's some uncertainty about the future of Laminas modules in general; maybe what will ultimately be needed is something of an entirely different shape... I'm still watching and waiting to see how everything ends up developing).

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