Latest Pareview report shows some doc formatting issues in MarketingCloudMessagesTest.

See the attached file.

CommentFileSizeAuthor
#3 marketing_cloud-MarketingCloudMessagesTest_prareview-2972899-3.patch6.85 KBjohn_a
#3 marketing_cloud-MarketingCloudMessagesTest_prareview-2972899-3.patch6.85 KBjohn_a
#2 marketing_cloud-MarketingCloudMessagesTest_prareview-2972899-2.patch6.53 KBjohn_a
MarketingCloudMessagesTest-pareview.txt4.53 KBjohn_a
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

john_a created an issue. See original summary.

john_a’s picture

john_a CreditAttribution: john_a as a volunteer commented
Status: Active » Needs review
FileSize
marketing_cloud-MarketingCloudMessagesTest_prareview-2972899-2.patch6.53 KB

Patch created, please review.

False negatives not addressed:

----------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
----------------------------------------------------------------------
48 | ERROR | [x] Namespaced classes/interfaces/traits should be
| | referenced with use statements
132 | ERROR | [x] Namespaced classes/interfaces/traits should be
| | referenced with use statements
----------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

john_a’s picture

john_a CreditAttribution: john_a as a volunteer commented
FileSize
marketing_cloud-MarketingCloudMessagesTest_prareview-2972899-3.patch6.85 KB
marketing_cloud-MarketingCloudMessagesTest_prareview-2972899-3.patch6.85 KB

Added new patch to fix the namespace issue

john_a’s picture

john_a CreditAttribution: john_a as a volunteer commented
Status: Needs review » Fixed

patch updated, reviewed and applied by self

john_a’s picture

john_a CreditAttribution: john_a as a volunteer commented
Status: Fixed » Closed (fixed)