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 without_patch_Maestro.png54.41 KBAanal.addweb
#3 with_patch_Maestro.png50.01 KBAanal.addweb
#3 dependency_Maestro.jpg302.34 KBAanal.addweb
#2 Update_dependencies_format_in_info.yml_3013095-2.patch5.44 KBchipway

Comments

chipway created an issue. See original summary.

chipway’s picture

chipway commented
Title: Must add prefix to dependencies » Must add prefix to dependencies + description end dot.
Assigned: chipway » Unassigned
Status: Needs work » Needs review
StatusFileSize
new Update_dependencies_format_in_info.yml_3013095-2.patch5.44 KB

Here is the patch. Please review.
I also added missing description end dots.

Aanal.addweb’s picture

Aanal.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new dependency_Maestro.jpg302.34 KB
new with_patch_Maestro.png50.01 KB
new without_patch_Maestro.png54.41 KB

@chipway, Thanks for providing the patch. Your patch is working fine for me. As per the mentioned changes, I matched it with Pareview and tested in simplytest.me over Extend page. PFA.

gogowitsch’s picture

gogowitsch commented

Patch from #3 looks good to me and applies fine.

  • _randy committed e18cfe6 on 8.x-2.x authored by chipway 
    Issue #3013095 by chipway, Aanal.addweb: Must add prefix to dependencies...
_randy’s picture

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

Status: Fixed » Closed (fixed)

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