Need to fix notices from CodeSniffer.

https://github.com/podarok/csua/pull/6

CommentFileSizeAuthor
#6 faq-codesniffer-2363389-5.patch72.22 KBsanchiz
#4 faq-codesniffer-2363389-4.patch76.58 KBsanchiz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sanchiz’s picture

sanchiz CreditAttribution: sanchiz commented

Need to fix notices from CodeSniffer.

sanchiz’s picture

sanchiz CreditAttribution: sanchiz commented

https://github.com/podarok/csua/pull/6

sanchiz’s picture

sanchiz CreditAttribution: sanchiz commented
Issue summary: View changes
sanchiz’s picture

sanchiz CreditAttribution: sanchiz commented
Status: Active » Needs review
FileSize
faq-codesniffer-2363389-4.patch76.58 KB

Initial patch.

Status: Needs review » Needs work

The last submitted patch, 4: faq-codesniffer-2363389-4.patch, failed testing.

sanchiz’s picture

sanchiz CreditAttribution: sanchiz commented
Status: Needs work » Needs review
FileSize
faq-codesniffer-2363389-5.patch72.22 KB

Re-roll according to module path.

  • podarok committed 5edec40 on 7.x-1.x authored by sanchiz 
    Issue #2363389 by sanchiz: Added Need to fix notices from CodeSniffer.
podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
CreditAttribution: podarok commented
Status: Needs review » Fixed

Thank You tons!!!
Merged

Status: Fixed » Closed (fixed)

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