Skip to content

Add doctrine/doctrine-bundle 3 support#1103

Closed
tomkalon wants to merge 3 commits intoSylius:1.14from
tomkalon:feature/add-doctrine-bundle-3-support
Closed

Add doctrine/doctrine-bundle 3 support#1103
tomkalon wants to merge 3 commits intoSylius:1.14from
tomkalon:feature/add-doctrine-bundle-3-support

Conversation

@tomkalon
Copy link
Copy Markdown

  • removed "lchrusciel/api-test-case" from dev dependency.

@Rafikooo Rafikooo linked an issue Nov 25, 2025 that may be closed by this pull request
@tomkalon tomkalon force-pushed the feature/add-doctrine-bundle-3-support branch 2 times, most recently from 7c644be to 75990af Compare November 27, 2025 14:24
…ith WebTestCase and refactor traits for shared logic.

- Update PHPUnit assertions to use stricter methods for status code verification.
- Adjust composer constraints for improved compatibility with future Symfony and Doctrine releases.
- Simplify and enhance XML and JSON test helpers for better reusability across controllers.
@tomkalon tomkalon force-pushed the feature/add-doctrine-bundle-3-support branch from 75990af to 6f10b05 Compare November 28, 2025 07:42
@tomkalon tomkalon force-pushed the feature/add-doctrine-bundle-3-support branch 14 times, most recently from bd61640 to 496af24 Compare November 28, 2025 11:21
@tomkalon tomkalon force-pushed the feature/add-doctrine-bundle-3-support branch from 496af24 to 05edc9c Compare November 28, 2025 11:22
},
"conflict": {
"doctrine/orm": "<2.18",
"doctrine/doctrine-bundle": "<2.0 || ^3.0",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No, you need to keep a conflict here.
It's an optional requirement.
"doctrine/doctrine-bundle": "<2.13",

NoResponseMate added a commit that referenced this pull request Dec 19, 2025
| Q               | A
| --------------- | -----
| Bug fix?        | no
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | fixes #1092
| License         | MIT

Replace #1103
@NoResponseMate
Copy link
Copy Markdown
Contributor

Since #1109 has been merged, I'm closing this 👋

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.

Allow doctrine/doctrine-bundle ^3

3 participants