Still I applied it + PHP CS Fixerphpdoc_align withvertical to get this result. Finally, an issue has been reported to Jetbrains so PHPStorm may supports this syntax:https://youtrack.jetbrains.com/issue/WI-76947/Allow-param-to-omit-the-type. 🤞

alexandre-daubois requested review fromOskarStark,chalasr,dunglas,jderusse,kbond,lyrixx,welcoMattic,xabbuh andyceruto ascode owners

April 8, 2024 13:59

carsonbot added the Status: Needs Review label

Apr 8, 2024

carsonbot added this to the7.1 milestone

Apr 8, 2024

alexandre-daubois force-pushed thedev-param-type branch from2965716 tof94d36eCompare

April 8, 2024 14:01

alexandre-daubois mentioned this pull request

Apr 8, 2024

Fix and improve overall PHPDocs#54489

Closed

GromNaN reviewed

Apr 8, 2024

View reviewed changes

src/Symfony/Bridge/Doctrine/Attribute/MapEntity.php OutdatedShow resolvedHide resolved

Remove redundant type in@param occurrences

7e232b2

alexandre-daubois force-pushed thedev-param-type branch fromf94d36e to7e232b2Compare

April 8, 2024 14:50

Copy link

Member

derrabus commentedApr 8, 2024

How's tool support for this syntax these days?

Copy link

MemberAuthor

alexandre-daubois commentedApr 9, 2024

PHPStorm struggles with this, hope my issue in Jetbrains' Youtrack gets some attention!
Psalm seems ok, as it didn't complain in this PR
PHPStan handles it well:https://phpstan.org/r/e471345c-3f0b-4a71-a593-c72639017700
phpDocumentor defines the type as optional so it's also fine
VSCode seems also fine with it

Am I missing something? I'd be happy to test more!

norkunas mentioned this pull request

Apr 12, 2024

[DEPRECATION] Param tags without namephpDocumentor/ReflectionDocBlock#362

Open

Copy link

Member

nicolas-grekas commentedApr 12, 2024

Please allow me to close here as this would need some automation to enforce in the long run. At the moment, this isn't worth the review+conflict-resolution time this will create IMHO.