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 commerce_vipps-info-3073313-2.patch291 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
commerce_vipps-info-3073313-2.patch291 bytes
Willtg’s picture

Willtg CreditAttribution: Willtg at Hook 42 commented
Status: Needs review » Needs work

The dependency name-spacing aligns with Drupal documentation standards. was unable to login with patch applied, marking as needs work.

  • alonaoneill authored 48fbf3c on 8.x-4.x 
    Issue #3073313 by alonaoneill: Capitalize the name of the module in ....
zaporylie’s picture

zaporylie
he/him
Primary language Polish
CreditAttribution: zaporylie for Ny Media AS commented
Status: Needs work » Fixed
Issue tags: +Amsterdam2019

Agree with @alonaoneill that capitalisation is good in this case so committed and pushed to 8.x-4.x. Thanks.

Status: Fixed » Closed (fixed)

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