-
Notifications
You must be signed in to change notification settings - Fork 155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Automatically remove useless comments #433
Automatically remove useless comments #433
Conversation
When issues are detected and classified as Magento2.Commenting.ClassAndInterfacePHPDocFormatting.ForbiddenTags or Magento2.Commenting.ClassAndInterfacePHPDocFormatting.InvalidDescription, these can be fixed by removing the offending comment.
@magento import PR to magento-commerce/magento-coding-standard repository |
@sinhaparul the Pull Request is successfully imported. |
@magento import PR to magento-commerce/magento-coding-standard repository |
@sidolov the Pull Request is successfully imported. |
Well, the whole point of this sniff was to enforce meaningful descriptions :) |
@orlangur, from the message:
So it does make sense to auto-remove it |
When issues are detected and classified as
Magento2.Commenting.ClassAndInterfacePHPDocFormatting.ForbiddenTags
orMagento2.Commenting.ClassAndInterfacePHPDocFormatting.InvalidDescription
, these can be fixed by removing the offending comment.