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!
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | node_type_permissions-info-3050627-2.patch | 403 bytes | alonaoneill |
Comments
Comment #2
alonaoneill commentedI uploaded a patch that Capitalize the name of the module and dependency namespacing in the .info.yml file.
Thanks!
Comment #3
amit.drupal commentedApply clearly.
Comment #4
avpadernoTo 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.
Comment #6
paul dudink commentedThanks for the help, the patch and the information people.
I just committed the patch, but I forgot to mention @alonaoneill in it. Sorry, will do next time (just found out about giving credit - I'm learning...).
Comment #7
paul dudink commentedRelease 8.x-1.0-alpha2 coming up.
Comment #8
paul dudink commented