Skip to content

Commit e28bf06

Browse files
rodrigoprimojrfnl
andcommitted
Apply suggestions from code review
Co-authored-by: Juliette <663378+jrfnl@users.noreply.github.com>
1 parent 009c701 commit e28bf06

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/Standards/Squiz/Tests/Commenting/PostStatementCommentUnitTest.inc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,12 +53,12 @@ $match = match($foo // comment
5353
1 => 1, // comment
5454
};
5555

56-
// Comments recognized as annotations by PHPCS.
56+
// Issue #560: Annotations should be reported separately and be non-auto-fixable as their meaning may change when moved.
5757
$a = 1; //@codeCoverageIgnore
5858
$b = 2; // @phpstan-ignore variable.undefined
5959
$c = 3; // @phpstan-ignore variable.undefined
6060
$d = 4; // @tabInsteadOfSpace
6161

62-
// Comments that include `@`, but are not recognized as annotations by PHPCS.
62+
// Comments that include `@`, but are not recognized as annotations by this sniff.
6363
$a = 1; // @ = add tag.
6464
$b = 2; // Some comment. // @username

src/Standards/Squiz/Tests/Commenting/PostStatementCommentUnitTest.inc.fixed

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,13 +58,13 @@ $match = match($foo // comment
5858
// comment
5959
};
6060

61-
// Comments recognized as annotations by PHPCS.
61+
// Issue #560: Annotations should be reported separately and be non-auto-fixable as their meaning may change when moved.
6262
$a = 1; //@codeCoverageIgnore
6363
$b = 2; // @phpstan-ignore variable.undefined
6464
$c = 3; // @phpstan-ignore variable.undefined
6565
$d = 4; // @tabInsteadOfSpace
6666

67-
// Comments that include `@`, but are not recognized as annotations by PHPCS.
67+
// Comments that include `@`, but are not recognized as annotations by this sniff.
6868
$a = 1;
6969
// @ = add tag.
7070
$b = 2;

0 commit comments

Comments
 (0)