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

aimeerae created an issue. See original summary.

aimeerae’s picture

Assigned: aimeerae » Unassigned
Status: Active » Needs review
StatusFileSize
new421 bytes

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

alonaoneill’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new13.51 KB

Code looks good, the name of the module is now capitalized on the Extend page. Screenshot provided."
The dependency namespacing aligns with Drupal documentation standards.
Marking as RTBC
Thank you for working on the module!

jrb’s picture

Status: Reviewed & tested by the community » Needs review

Is each word actually supposed to be capitalized? A standard in Drupal in many places seems to be to capitalize the first word, but not all. Just on the first page of results here, there are 3 with lowercase words:

https://www.drupal.org/project/project_module

Administration menu
CKEditor - WYSIWYG HTML editor
XML sitemap

  • jrb committed da8d428 on 8.x-1.x authored by aimeerae
    Issue #3031166 by aimeerae, alonaoneill, jrb: Capitalize the name of the...
jrb’s picture

Status: Needs review » Fixed

I checked core modules and confirmed that those modules have each word in their names capitalized. I've committed this patch and will create a new alpha. Thanks!

jrb’s picture

Status: Fixed » Closed (fixed)