Class properties that have types should not require a DocBlock

[aligent-lturner]

Fixes #404

If a class property has a type, then there should not be a requirement for a DocBlock. A DocBlock can still be added if desired - this change simply removes the warning for a missing comment block when the property has a defined type.

Note - this does not validate that the type is valid, only that it is not specifically invalid in the same way as https://github.com/magento/magento-coding-standard/blob/develop/Magento2/Sniffs/Annotation/MethodArgumentsSniff.php

Additional: This issue was originally raised against the documentation page (https://github.com/AdobeDocs/commerce-php/issues/23), and was suggested to be raised directly here. The documentation would need to be updated to reflect that DocBlocks are only required for properties without a type.

[aligent-lturner]

Failing tests do not appear to be related to this change, but a missing Symfony class:

Error: Class "Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator" not found

[torhoehn]

@sivaschenko Would it be possible to merge this please?

[aligent-lturner]

hi @aligent-lturner, Could you please resolve conflicts?

@ihor-sviziev conflict has been resolved

[sinhaparul]

@magento import PR to magento-commerce/magento-coding-standard repository

[m2-github-services]

@sinhaparul the Pull Request is successfully imported.

[fredden]

See also #476

[ihor-sviziev]

@sinhaparul @sidolov could you please merge it?