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

feat: compatibility with doctrine orm 3.0 minimum #99

Merged

Conversation

monpetitplacement
Copy link

@monpetitplacement monpetitplacement commented Apr 18, 2024

This PR updates the extension to be compatible with Doctrine 3.0, which is now the minimum required version. I had to remove support for ArrayType and ObjectType, as they are no longer natively supported by Doctrine.

All tests pass.

@Gwemox
Copy link
Contributor

Gwemox commented Apr 18, 2024

image

@monpetitplacement monpetitplacement changed the base branch from master to 3.x April 22, 2024 10:17
@Gwemox
Copy link
Contributor

Gwemox commented Apr 22, 2024

@x86demon can you rebase branch 3.x with master ?

@x86demon
Copy link
Collaborator

Hi @Gwemox. master mostly contains in-development features that are planned for next release. What functionality do you need in 3.x? Maybe I can cherry-pick some commits or adopt PR for 3.x?

@Gwemox
Copy link
Contributor

Gwemox commented Apr 24, 2024

Hi @Gwemox. master mostly contains in-development features that are planned for next release. What functionality do you need in 3.x? Maybe I can cherry-pick some commits or adopt PR for 3.x?

Add CI?

@x86demon
Copy link
Collaborator

CI was ported to 2.x and 3.x

@Gwemox Gwemox force-pushed the feat/upgrade-doctrine-orm-3 branch 2 times, most recently from e409b58 to f6d5d2f Compare April 24, 2024 13:33
@Gwemox Gwemox force-pushed the feat/upgrade-doctrine-orm-3 branch from f6d5d2f to b221630 Compare April 24, 2024 13:53
@Gwemox
Copy link
Contributor

Gwemox commented Apr 24, 2024

It's OK, you can check the code.

@Gwemox
Copy link
Contributor

Gwemox commented May 15, 2024

Up @x86demon ? 👼

@x86demon
Copy link
Collaborator

Hi @Gwemox. Thank you for the PR
I'll put it in the queue for review and will do my best to review it in one of upcoming sprints.

@yakobe yakobe mentioned this pull request Jun 25, 2024
@yakobe
Copy link

yakobe commented Jun 25, 2024

Would be great if this could get merged and released. It's the last compatibility blocker for me 😃

@x86demon x86demon merged commit f44d9cf into oroinc:3.x Jul 1, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants