magento-coding-standard icon indicating copy to clipboard operation
magento-coding-standard copied to clipboard

Published 20 hours ago • verified publisher icon magento

@author MUST NOT be used.

Open arushibansal013 opened this issue 4 years ago • 4 comments

Rule

@author MUST NOT be used

Reason

As per magento DocBlock document (https://devdocs.magento.com/guides/v2.3/coding-standards/docblock-standard-general.html#documentation-space)

Structure of documentation space @author ,@category, @package, and @subpackage MUST NOT be used. Documentation is organized with the use of namespaces.

Implementation

Need to add @author in $forbiddenTags

arushibansal013 avatar Jan 09 '20 09:01 arushibansal013

@arushibansal013 I have created a PR (#204) that adds the ability to use phpcbf to remove automatically. Is it worth me adding @author in the same PR?

cjnewbs avatar May 17 '21 17:05 cjnewbs

Hi @arushibansal013 @fredden, Could you please explain why shouldn't we allow using the @author tag in the coding standards? As for me - it might be relevant only to Magento / Adobe Commerce core but not relevant for all custom extensions.

ihor-sviziev avatar Jun 13 '22 14:06 ihor-sviziev

It looks like the rule was introduced here: https://github.com/magento/devdocs/commit/4165be844c0f96bdcc41b2f952668579266f1a6e

fredden avatar Jun 13 '22 15:06 fredden

I believe all the mentioned tags must not exist in the Magento core, but it should be allowed for the custom extensions.

Am I right, @Vinai @sivaschenko ?

What I mean - we should move this rule to the following rule set https://github.com/magento/magento-coding-standard/blob/develop/Magento2Framework/ruleset.xml and add what you just suggest editors. Does it make sense?

ihor-sviziev avatar Jun 14 '22 19:06 ihor-sviziev