You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
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?
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.
The text was updated successfully, but these errors were encountered: