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
new344 bytes

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

amit.drupal’s picture

Status: Needs review » Reviewed & tested by the community

Apply clearly.

avpaderno’s picture

To give more context to the project maintainers: User interface standards / Interface text (Module and theme names) says that module and theme names should use title case capitalization.

avpaderno’s picture

Status: Reviewed & tested by the community » Needs work

The module name should be written using the title case capitalization, for which not all the words are capitalized.

alonaoneill’s picture

Status: Needs work » Needs review
StatusFileSize
new344 bytes

Made changes.

avpaderno’s picture

Status: Needs review » Reviewed & tested by the community

leon kessler’s picture

Status: Reviewed & tested by the community » Fixed

Pushed to 8.x-1.x. Thanks and sorry for the delay!

Status: Fixed » Closed (fixed)

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