While looking at the code, I found some points for improvement.

CommentFileSizeAuthor
#2 social-demo-touch-up-2834600-2.patch1.56 KBSutharsan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sutharsan created an issue. See original summary.

Sutharsan’s picture

Sutharsan CreditAttribution: Sutharsan as a volunteer commented
Status: Active » Needs review
FileSize
social-demo-touch-up-2834600-2.patch1.56 KB
  • Replaces "\" with "\\" in class names strings.
  • Improves readability of drush_print() command
  • Removes empty .module file. This file is no longer required in D8.
jaapjan’s picture

jaapjan CreditAttribution: jaapjan commented
Status: Needs review » Fixed

Thanks for the patch. Just merged it.

  • 1b3a73c committed on 8.x-1.x 
    Merge pull request #209 from goalgorilla/feature/2834600

Issue #2834600...
  • jaapjan committed fa8cb84 on 8.x-1.x authored by Sutharsan
    Issue #2834600 by Sutharsan: Touch-ups demo module

Status: Fixed » Closed (fixed)

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