Please apply new {project}:{module} format for dependencies in info.yml filehttps://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-about-your-module-with-an-infoyml-file

It is supported since 8.0 and 7.40 Change Record: Project namespaces can now be added for module dependencies, and is now a Best Practice Define project dependencies in core module .info files).

It is useful to install/update via composer. So sooner is better.

+ Capitalize module name following Coding Standards.

Hope this helps.

CommentFileSizeAuthor
#3 Screen Shot 2018-09-26 at 2.50.11 PM.png6.35 KBalonaoneill
#2 Update_dependencies_format_in_info.yml_3001155-2.patch474 byteschipway

Comments

chipway created an issue. See original summary.

chipway’s picture

chipway commented
Assigned: chipway » Unassigned
Status: Needs work » Needs review
StatusFileSize
new Update_dependencies_format_in_info.yml_3001155-2.patch474 bytes

Here is the patch. Please review.

alonaoneill’s picture

alonaoneill commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Screen Shot 2018-09-26 at 2.50.11 PM.png6.35 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 module!

mohammed j. razem’s picture

mohammed j. razem
Location Santa Clara, CA
commented

Reviewing this now.

mohammed j. razem’s picture

mohammed j. razem
Location Santa Clara, CA
commented
Status: Reviewed & tested by the community » Fixed

Committed to dev branch.

Will be working on a release very soon in the next hours.

Status: Fixed » Closed (fixed)

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