When running code sniffer against the module the tests fail. Attached is the report.

CommentFileSizeAuthor
#2 commerce_popup_cart-2751381-phpcs.patch10.54 KBskwashd
commerce_popup_cart-codesniffer-report.txt15.3 KBskwashd

Comments

skwashd created an issue. See original summary.

skwashd’s picture

skwashd commented
Status: Active » Needs review
StatusFileSize
new commerce_popup_cart-2751381-phpcs.patch10.54 KB

The attached patch results in code sniffer passing.

Everything is pretty straight forward. I removed the documentation from the hoverIntent implementation as it is available in the jQuery lib docs if anyone needs to look it up.

3cwebdev’s picture

3cwebdev commented

Patch applied to dev.

skwashd’s picture

skwashd commented
Status: Needs review » Fixed

The patch has been committed. Marking as fixed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.