Skip to content

Conversation

@VincentLanglet
Copy link
Contributor

@VincentLanglet VincentLanglet commented Dec 7, 2025

Q A
Branch? 7.3
Bug fix? yes
New feature? no
Deprecations? no
Issues Fix #62636
License MIT

The PR on 6.4 is #62686

I'm not sure how you want to deal with the merges since there will be conflict when merging up 6.4 into 7.3 so I opened both PR to show the changes

@carsonbot carsonbot added this to the 8.1 milestone Dec 7, 2025
@carsonbot carsonbot changed the title Add failing test for 62636 Add failing test for 62636 Dec 7, 2025
@carsonbot carsonbot changed the title Add failing test for 62636 [PropertyInfo] Add failing test for 62636 Dec 7, 2025
@VincentLanglet VincentLanglet changed the base branch from 8.1 to 7.3 December 7, 2025 22:00
@VincentLanglet VincentLanglet changed the title [PropertyInfo] Add failing test for 62636 [PropertyInfo] Fix PseudoType support in PhpDocTypeHelper Dec 7, 2025
@VincentLanglet
Copy link
Contributor Author

Milestone should be edited to 7.3 @OskarStark.

Also I dunno how you want to handle this fix since the PR from 6.4 (#62686) will create conflict, so I opened the fix on 7.3 too to show the expected resutl.

nicolas-grekas added a commit that referenced this pull request Dec 10, 2025
…(VincentLanglet)

This PR was squashed before being merged into the 6.4 branch.

Discussion
----------

[PropertyInfo] Fix PseudoType support in PhpDocTypeHelper

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #62636
| License       | MIT

The PR on 7.3 is #62684

I'm not sure how you want to deal with the merges since there will be conflict when merging up 6.4 into 7.3 so I opened both PR to show the changes

Commits
-------

e2b7844 [PropertyInfo] Fix PseudoType support in PhpDocTypeHelper
@nicolas-grekas nicolas-grekas modified the milestones: 8.1, 7.3 Dec 10, 2025
@nicolas-grekas
Copy link
Member

Thank you @VincentLanglet.

@nicolas-grekas nicolas-grekas merged commit 5b76ddf into symfony:7.3 Dec 10, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants