Apply new {project}:{module} format for dependencies in info.yml https://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-about-your-module-with-an-infoyml-file
My 2 cents.

CommentFileSizeAuthor
#2 Update_dependencies_format_in_info.yml_2983969-2.patch319 byteschipway
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chipway created an issue. See original summary.

chipway’s picture

chipway CreditAttribution: chipway at Chipway commented
Assigned: chipway » Unassigned
Status: Needs work » Needs review
FileSize
Update_dependencies_format_in_info.yml_2983969-2.patch319 bytes

Please review

chipway’s picture

chipway CreditAttribution: chipway at Chipway commented

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).

alonaoneill’s picture

alonaoneill CreditAttribution: alonaoneill at Hook 42 commented
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!

  • g089h515r806 committed 0683626 on 8.x-1.x authored by chipway 
    Issue #2983969 by chipway: Apply new {project}:{module} format for...
g089h515r806’s picture

g089h515r806 CreditAttribution: g089h515r806 commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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