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 media_entity_audio-info-3019479-2.patch313 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
media_entity_audio-info-3019479-2.patch313 bytes

I uploaded a patch that Capitalize the name of the module in the .info.yml file.
Thanks!

Status: Needs review » Needs work

The last submitted patch, 2: media_entity_audio-info-3019479-2.patch, failed testing. View results

alonaoneill’s picture

alonaoneill CreditAttribution: alonaoneill at Hook 42 commented
Status: Needs work » Needs review
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 and capitalization aligns with Drupal documentation standards. Marking as RTBC Thank you for working on the module!

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed.

  • Berdir committed 800a787 on 8.x-2.x authored by alonaoneill 
    Issue #3019479 by alonaoneill: Capitalize the name of the module in ....

Status: Fixed » Closed (fixed)

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