search

w3guy / persist-admin-notices-dismissal

Simple plugin that persists dismissal of admin notices across pages in WordPress dashboard.
http://w3guy.com/wordpress-admin-notices-dismissible/
90 stars 22 forks source link

WooCommerce WPCS #40

Closed Surbma closed 8 months ago

Surbma commented 3 years ago

The WooCommerce CS is more stricter, than normal WPCS. It is giving errors for some @ tags in the comments:

Please consider to change them to comply with WCCS also. This modification is required to validate the code for WooCommerce.com repository.

Example change in the persist-admin-notices-dismissal.php file: @author Collins Agbonghama, Andy Fragen -> Author: Collins Agbonghama, Andy Fragen

Thank you if you consider this change in the code. That would make PAnD code valid for WC standards as well.

afragen commented 3 years ago

Why don't you simply ignore the vendor folder from your phpcs check?

Surbma commented 3 years ago

I can ignore, but WooCommerce.com doesn't ignore, so I have to deal with it. Everything that goes to WooCommerce.com Marketplace have to pass these coding standards.