Edit report at http://pear.php.net/bugs/bug.php?id=14002&edit=1

ID: 14002
Updated by: gsherwood@squiz.net
Reported By: bender at casadebender dot com
Summary: Add mechanism to suppress error/warning due to failed
-Status: Assigned
+Status: Closed
Type: Feature/Change Request
Package: PHP_CodeSniffer
Package Version: CVS
PHP Version: Irrelevant
Assigned To: squiz
Roadmap Versions:
New Comment:

-Status: Assigned
+Status: Closed
This bug has been fixed in CVS.

If this was a documentation problem, the fix will appear on
pear.php.net by the end of next Sunday (CET).

If this was a problem with the pear.php.net website, the change should
be live shortly.

Otherwise, the fix will appear in the package's next release.

Thank you for the report and for helping us make PEAR better.

I've added this feature but used different tags so they match the ones
in PHPUnit. I've also decided not to print any messages about
suppressing errors and warnings, but they may be added later if it
becomes useful for people to know.

Use the tags like this:
// @codingStandardsIgnoreStart
echo "no errors here";
// @codingStandardsIgnoreEnd

Previous Comments:

[2008-05-29 18:18:01] squiz

Thanks. I didn't notice it didn't appear in the bug list.


[2008-05-29 11:24:58] bender

Both Request #14000 and Request #14002 have been closed as dups of each
other. Request #14000 somehow messed up the submitter authentication and
is not visible to normal bugs browsers. This ticket however is visible,
so it should probably be the one to remain open until the patch is
applied or rejected.


[2008-05-29 06:01:14] squiz

Duplicate of Request #14000, which contains the same patch.


[2008-05-28 01:40:34] bender

Sometimes rules are meant to be broken. This patch allows an error or
warning to be suppressed by adding a specially formatted comment to the
line proceeding the offending code.

A comment of the form "//@SUPPRESS" triggers the suppression. If the
verbosity of CodeSniffer is greater than 0 (default), a message will be
output to stdout announcing the error or warning that has been


Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
grouppear-bugs @
postedJul 3, '09 at 6:22a
activeJul 3, '09 at 6:22a

1 user in discussion

Gsherwood: 1 post



site design / logo © 2021 Grokbase