Do not require @var tag if a property has typehint

The attached patch adds a check in case no @var is found to determine whether the token before the variable for which we're looking at the doc comment is a string. This is the token type for a type annotation. If no type is specified for the property then the previous token would be T_PROTECTED, T_PUBLIC or T_PRIVATE.

Updated the patch to do the same. The previous patch was written in a version before some dead code (that I now relied on) was removed. The new implementation is also more robust in how it finds the type if it exists (being more resistant to whitespace changes).

Given the rewrite no interdiff is provided.

Nice, but needs tests

Opened a PR in https://github.com/pfrenssen/coder/pull/164. Added tests

Looking good, thanks! I think we need to wait for #3238192: Allow omitting @var for strictly typed class properties though before this can be merged.

Thanks, makes sense to me, just left a minor improvement comment on the pull request.

We don't need to wait for official coding standard approval in Coder in non-controversial cases such as this, so once ready we can merge immediately :)

Improved the PR and merged it. Thank you all!

I was just looking for something like this. Thanks all!

Thank you!