Open m2-assistant[bot] opened 1 year ago
Hi @engcom-Dash. Thank you for working on this issue. In order to make sure that issue has enough information and ready for development, please read and check the following instruction: :point_down:
Area: XXXXX
label to the ticket, indicating the functional areas it may be related to.2.4-develop
branch@magento give me 2.4-develop instance
to deploy test instance on Magento infrastructure. 2.4-develop
branch, please, add the label Reproduced on 2.4.x
.Hello @fredden,
Thanks for the report and collaboration!
We can see the PR related to this GitHub issue contains the improvement in the DocBlock as mentioned in the below documentation:
https://developer.adobe.com/commerce/php/coding-standards/docblock/#structure-of-documentation-space
https://github.com/author ,@category, @Package, and @subpackage MUST NOT be used. Documentation is organized with the use of namespaces.
Hence confirming this issue for further processing.
Thanks
:white_check_mark: Jira issue https://jira.corp.adobe.com/browse/AC-8370 is successfully created for this GitHub issue.
:white_check_mark: Confirmed by @engcom-Dash. Thank you for verifying the issue.
Issue Available: @engcom-Dash, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself.
This issue is automatically created based on existing pull request: magento/magento2#36988: Remove forbidden
@author
tag fromMagento_Catalog
(part 2)Description
According to https://devdocs.magento.com/guides/v2.4/coding-standards/docblock-standard-general.html#documentation-space, the
@author
tag is not permitted in Magento. This pull request removes this tag from (some of) theMagento_Catalog
module. Given there are so many instances of this tag, I've opened a small pull request to get the process started. I expect that the linter will force me to fix several other coding standards violations on the way, so having a smaller pull request means that task is easier to manage. I plan to open more pull requests to tackle the other instances of this tag.See also https://github.com/magento/magento-coding-standard/pull/382 and https://github.com/magento/magento-coding-standard/issues/167
See also https://github.com/magento/magento2/pull/36304, https://github.com/magento/magento2/pull/36976, https://github.com/magento/magento2/pull/36977, https://github.com/magento/magento2/pull/36978, https://github.com/magento/magento2/pull/36979, https://github.com/magento/magento2/pull/36980, https://github.com/magento/magento2/pull/36981, https://github.com/magento/magento2/pull/36987
Manual testing scenarios
There are not code changes in this pull request. This pull request only removes forbidden comments.
Contribution checklist