Merge pull request #1 from CommonGateway/feature/mf-78/notification-a… #16
Annotations
10 errors and 2 warnings
build:
src/CustomerNotificationsBundle.php#L5
The tag in position 1 should be the @category tag
|
build:
src/CustomerNotificationsBundle.php#L5
Content of the @author tag must be in the form "Display Name <[email protected]>"
|
build:
src/CustomerNotificationsBundle.php#L6
The tag in position 2 should be the @author tag
|
build:
src/CustomerNotificationsBundle.php#L8
The tag in position 3 should be the @license tag
|
build:
src/Service/InstallationService.php#L2
Missing file doc comment
|
build:
src/Service/InstallationService.php#L12
Tag value for @author tag indented incorrectly; expected 1 spaces but found 2
|
build:
src/Service/InstallationService.php#L89
There must be no blank line following an inline comment
|
build:
src/Service/InstallationService.php#L101
No space found before comment text; expected "// This is the place where you can add or change Installation data from/for this bundle or other required bundles." but found "//This is the place where you can add or change Installation data from/for this bundle or other required bundles."
|
build:
src/Service/InstallationService.php#L102
No space found before comment text; expected "// Note that in most cases it is recommended to use .json files in the Installation folder instead, if possible." but found "//Note that in most cases it is recommended to use .json files in the Installation folder instead, if possible."
|
build:
src/Service/InstallationService.php#L102
There must be no blank line following an inline comment
|
build:
src/Service/NotificationsService.php#L160
Comment refers to a TODO task "Maybe we could/should support multiple Sources instead of one?"
|
build:
src/Service/NotificationsService.php#L185
Line exceeds 125 characters; contains 146 characters
|