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

chore: upgrade to doctrine/orm 3 #5365

Draft
wants to merge 4 commits into
base: devel
Choose a base branch
from
Draft

Conversation

usu
Copy link
Member

@usu usu commented Jun 15, 2024

In the meantime, orm-3 compatibility was merged into main branch of https://github.com/doctrine-extensions/DoctrineExtensions

This PR is opened as draft to test orm-3 and to be ready, once the next version of DoctrineExtension is released

@usu usu added the deploy! Creates a feature branch deployment for this PR label Jun 15, 2024
Copy link

github-actions bot commented Jun 15, 2024

Feature branch deployment currently inactive.

If the PR is still open, you can add the deploy! label to this PR to trigger a feature branch deployment.

@usu
Copy link
Member Author

usu commented Jun 18, 2024

Also blocked by doctrine/migrations#1406

@@ -299,23 +326,6 @@ public function testNormalizeDoesntReplaceWhenNoFilters() {
$this->shouldNotReplaceChildren($result);
}

public function testNormalizeDoesntReplaceWhenTargetEntityIsMissing() {
Copy link
Contributor

Choose a reason for hiding this comment

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

why did you delete this test?

@usu usu temporarily deployed to feature-branch July 9, 2024 19:05 — with GitHub Actions Inactive
@usu
Copy link
Member Author

usu commented Jul 9, 2024

Blocked by doctrine/migrations#1406

@usu usu temporarily deployed to feature-branch July 9, 2024 19:13 — with GitHub Actions Inactive
@BacLuc BacLuc removed the deploy! Creates a feature branch deployment for this PR label Jul 19, 2024
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.

2 participants