Capitalize the name of the module, because module names are proper nouns.
https://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-...
Patch to follow, thanks!

CommentFileSizeAuthor
#2 twig_vardumper-info-3075292-2.patch316 bytesalonaoneill
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alonaoneill created an issue. See original summary.

alonaoneill’s picture

alonaoneill CreditAttribution: alonaoneill at Hook 42 commented
Assigned: alonaoneill » Unassigned
Status: Active » Needs review
FileSize
twig_vardumper-info-3075292-2.patch316 bytes
Willtg’s picture

Willtg CreditAttribution: Willtg at Hook 42 commented
Status: Needs review » Reviewed & tested by the community

Code looks good. The dependency name-spacing aligns with Drupal documentation standards. Applied when tested with STM.
Marking as RTBC
Thank you for working on the module!

  • keopx committed 867669d on 8.x-1.x 
    Issue #3075292: Change name from vardumper to VarDumper.
keopx’s picture

keopx
Primary language Basque
Location Basque Country
CreditAttribution: keopx at Isarea commented
Status: Reviewed & tested by the community » Fixed

  • keopx committed 6f288dc on 8.x-1.x 
    Issue #3075292: Change name from vardumper to VarDumper.

Status: Fixed » Closed (fixed)

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