serializer icon indicating copy to clipboard operation
serializer copied to clipboard

Enable RequireOneLinePropertyDocComment sniff

Open simPod opened this issue 4 years ago • 2 comments

Q A
Bug fix? no
New feature? no
Doc updated no
BC breaks? no
Deprecations? no
Tests pass?
Fixed tickets #...
License MIT

simPod avatar Jan 11 '21 07:01 simPod

TBH i do not see a value in this, it was intentionally disabled that sniff (and phpstorm automatically uses the multiline version...)

goetas avatar Jan 11 '21 11:01 goetas

I thought that for easier maintenance it would be best to comply with Doctrine standard. That said removing all custom settings, exclusions etc.

Eg. this sniff saves a lot of file vertical space. Phpstorm has somehow outdated file templates, that's true.

In last few years phpdoc are started disappearing rapidly as we have param, return and property types. Will get event better with annotations and union types, mb even generics.

simPod avatar Jan 11 '21 12:01 simPod