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 with_patch_Changed Fields API.png82.74 KBAanal.addweb
#3 without_patch_Changed Fields API.png77.57 KBAanal.addweb
#3 dependency_Changed Fields API.png58.2 KBAanal.addweb
#2 Update_dependencies_format_in_info.yml_3018518-2.patch1.86 KBchipway

Comments

chipway created an issue. See original summary.

chipway’s picture

chipway commented
Title: All dependencies must be prefixed » All dependencies must be prefixed + misc CS
Assigned: chipway » Unassigned
Status: Needs work » Needs review
StatusFileSize
new Update_dependencies_format_in_info.yml_3018518-2.patch1.86 KB

Here is the patch. Please review.
I also
added missing description end dot.
capitalized some module name per coding standards.

Aanal.addweb’s picture

Aanal.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new dependency_Changed Fields API.png58.2 KB
new without_patch_Changed Fields API.png77.57 KB
new with_patch_Changed Fields API.png82.74 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.

  • Loparev committed f9015a9 on 8.x-3.x authored by chipway 
    Issue #3018518 by chipway, Aanal.addweb: All dependencies must be...
loparev’s picture

loparev commented

Thank you both

loparev’s picture

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

Status: Fixed » Closed (fixed)

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