Capitalize the name of the module, because module names are proper nouns.
Also according to Drupal standards, modules should include dependencies in the .info.yml file.
Dependencies should be namespaced in the format {project}:{module}, where {project} is the project name as it appears in the Drupal.org URL (e.g. drupal.org/project/views) and {module} is the module's machine name.
https://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-...
Patch to follow, thanks!

CommentFileSizeAuthor
#2 dtuber-info-3062333-2.patch329 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
dtuber-info-3062333-2.patch329 bytes

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

Zahor’s picture

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

Looks good

JayKandari’s picture

JayKandari
Primary language Hindi
CreditAttribution: JayKandari as a volunteer and at QED42 commented

Thank you @alonaoneill & @Zahor for the fix.

  • JayKandari committed 627ac9e on 8.x-1.x authored by alonaoneill 
    Issue #3062333 by alonaoneill, Zahor, JayKandari: Capitalize the name of...
JayKandari’s picture

JayKandari
Primary language Hindi
CreditAttribution: JayKandari as a volunteer and at QED42 commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x-1.x branch.

Status: Fixed » Closed (fixed)

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