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 will be useful to install/update via composer. So sooner is better.
Hope this helps.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chipway created an issue. See original summary.

chipway’s picture

Assigned: chipway » Unassigned
Status: Needs work » Needs review
FileSize
400 bytes

Here is the patch. Please review.

alonaoneill’s picture

Status: Needs review » Reviewed & tested by the community

Code looks good. The dependency namespacing aligns with Drupal documentation standards.
Patch applied via simplytest.me
Marking as RTBC
Thank you for working on module!

  • RajabNatshah committed 5953ca1 on 8.x-3.x authored by chipway
    Issue #3000231 by chipway, alonaoneill: Update Dependencies for composer
    
Rajab Natshah’s picture

Assigned: Unassigned » Mohammed J. Razem
Status: Reviewed & tested by the community » Needs review
Rajab Natshah’s picture

Assigned: Mohammed J. Razem » Unassigned
Rajab Natshah’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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