squizlabs
Squiz.Commenting.VariableComment doesn't allow @deprecated in variable docblock
It would be nice to be able to specify @deprecated for a class property, but currently this is banned by the Squiz VariableComment sniff.
I've read over some older github issues created for similar requests, and I can understand why you would be reluctant to modify the Squiz standard to accommodate additional tags. In this case, it would be nice if the processMemberVar() method in the relevant class could be broken up so that it is easier to override just the relevant part that checks for allowed tags. Alternatively, maybe the list of allowed tags can be made configurable?
Regardless of which outcome is chosen, I'd be happy to work on a patch to make this possible.
Thanks for all the great work you've all put in to PHPCS over the years, I'm incredibly grateful.
Related to #2222 which should make this a whole lot easier (and which I'm already working on behind the scenes in combination with #2189) ;-)
:+1: I'll keep an eye on those issues. Thanks for all your hard work in the latest release by the way!
this was a surprise to me today. seems this was filed before 3.x was released and before some merges that could support it. Is it something that could happen now?
