All dependencies must be prefixed by project name, So please apply new {project}:{module} format for dependencies in info.yml 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 DrupalCi to download the right dependencies and Installation profiles to work well. So sooner is better.
Hope this helps.

CommentFileSizeAuthor
#3 dependency_CKEditor Find_Replace.png39.13 KBzalak.addweb
#2 Update_dependencies_format_in_info.yml_3009721-2.patch356 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_3009721-2.patch356 bytes

Here is the patch. Please review.

zalak.addweb’s picture

zalak.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new dependency_CKEditor Find_Replace.png39.13 KB

@chipway, Thanks for putting an effort to make this patch works, It is working fineas per the standards you mentioned via attached links. I verify it with the use of pareview & attached the screenshot for the same.

  • dhansen committed 6762521 on 8.x-1.x authored by chipway 
    Issue #3009721 by chipway, dhansen, zalak.addweb: Apply new {project}:{...
dhansen’s picture

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

Patch added, thanks for your contribution.

I also went ahead and dropped the drupal:editor dependency since drupal:ckeditor already requires it.

Status: Fixed » Closed (fixed)

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