Skip to content

Comments

Restore some more lines from v6 of the bridge#2092

Merged
greg0ire merged 1 commit intodoctrine:2.17.xfrom
greg0ire:broken-compat-2
Oct 11, 2025
Merged

Restore some more lines from v6 of the bridge#2092
greg0ire merged 1 commit intodoctrine:2.17.xfrom
greg0ire:broken-compat-2

Conversation

@greg0ire
Copy link
Member

@greg0ire greg0ire commented Oct 10, 2025

I am not sure how I messed this up, but I managed to omit these lines when copy/pasting from the diff I put at #2082 (comment)

Reported in #2082 (comment) by @vitek-rostislav

How can I test this?

composer config repositories.greg0ire vcs https://github.com/greg0ire/DoctrineBundle
composer require doctrine/doctrine-bundle "dev-broken-compat-2 as 2.17.1"

@vitek-rostislav can you please confirm it fixes your issue?

I am not sure how I messed this up, but I managed to omit these lines
when copy/pasting from the diff I put at doctrine#2082 (comment)
@vitek-rostislav
Copy link

Hi, yes, I confirm it works now with my annotation mappings 👌

@greg0ire greg0ire added the Bug label Oct 10, 2025
@greg0ire greg0ire added this to the 2.17.2 milestone Oct 10, 2025
@greg0ire greg0ire merged commit 2543589 into doctrine:2.17.x Oct 11, 2025
13 checks passed
@greg0ire greg0ire deleted the broken-compat-2 branch October 11, 2025 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants