All dependencies must be prefixed by project name, So please apply new {project}:{module} format for dephttps://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-... 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 for Installation profiles to work well. So sooner is better.
Hope this helps.

CommentFileSizeAuthor
#2 Update_dependencies_format_in_info.yml_3003266-2.patch771 byteschipway

Comments

chipway created an issue. See original summary.

chipway’s picture

chipway commented
Title: Apply new format for dependencies » Apply new format for dependencies + end line
Assigned: chipway » Unassigned
Status: Needs work » Needs review
StatusFileSize
new Update_dependencies_format_in_info.yml_3003266-2.patch771 bytes

Here is the patch. Please review.

I also added the missing last empty line.

alonaoneill’s picture

alonaoneill 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!

  • adci_contributor committed 8b81114 on 8.x-1.x 
    Issue #3003266 by chipway, alonaoneill, adci_contributor: Apply new...
adci_contributor’s picture

adci_contributor commented
Status: Reviewed & tested by the community » Fixed

Thanks, chipway and alonaoneill for the patch and review, your work is appreciated. We applied your patch code with some additional code style changes.

Status: Fixed » Closed (fixed)

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