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-...
and
"Capitalize the name of the module, because module names are proper nouns."
https://www.drupal.org/docs/develop/documenting-your-project/help-text-s...
Patch to follow, thanks!
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | Screen Shot 2018-10-22 at 9.16.43 AM.png | 10.51 KB | alonaoneill |
| #2 | ebrsww-info-file-3007174-2.patch | 435 bytes | volkswagenchick |
Comments
Comment #2
volkswagenchickI uploaded a patch that addresses the changes necessary in the .info file. Thanks!
Comment #3
alonaoneill commentedCode 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 module!
Comment #4
legolasboYup, patch looks good to me. I'll commit and backport it.
Comment #7
legolasboCommitted and pushed to 8.x-2.x and 8.x-1.x. Thank you both for your contributions! :)
Comment #8
legolasboClosed #3007523: Capitalize the name of the module in .info.yml file as a duplicate