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!

Comments

alonaoneill created an issue. See original summary.

alonaoneill’s picture

Assigned: alonaoneill » Unassigned
Status: Active » Needs review
StatusFileSize
new469 bytes

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

Aanal.addweb’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new26.1 KB
new26.09 KB
new31.63 KB

@alonaoneill, Thanks for providing the patch. Your patch is working fine for me. As per the mentioned changes, I matched it with Pareview and tested in simplytest.me over Extend page. PFA.

  • alansaviolobo committed 0bdbc9d on 8.x-1.x
    Issue #3019164 by alonaoneill, Aanal.addweb: Capitalize the name of the...
alansaviolobo’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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