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!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alonaoneill created an issue. See original summary.

alonaoneill’s picture

Assigned: alonaoneill » Unassigned
Status: Active » Needs review
FileSize
818 bytes

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

krina.addweb’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
25.02 KB
39.26 KB

@alonaoneill,Thanks for the patch. Your patch is working fine as I tested it in my local over Extend page. PFA

stijn.blomme’s picture

FileSize
851 bytes

Thank you for the patch & for testing!

I altered the module description to something that describes the module better.
I guess this is ready to be committed to dev

stijn.blomme’s picture

Status: Reviewed & tested by the community » Fixed

  • stijn.blomme authored ef010c2 on 8.x-1.x
    Issue #3063152 by alonaoneill, stijn.blomme, krina.addweb: Capitalize...

Status: Fixed » Closed (fixed)

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