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
#5 with_patch_Invite.png20.25 KBAanal.addweb
#5 without_patch_Invite.png19.83 KBAanal.addweb
#5 dependency_Invite2.png32.08 KBAanal.addweb
#5 dependency_Invite.png31.76 KBAanal.addweb
#4 Update_dependencies_format_in_info.yml_3015053-4.patch808 byteschipway
#2 Update_dependencies_format_in_info.yml_3015053-2.patch0 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_3015053-2.patch0 bytes

Here is the patch. Please review.

Aanal.addweb’s picture

Aanal.addweb commented
Status: Needs review » Needs work

@chipway, It seems your patch is wrongly uploaded. Can you please do some needful?

chipway’s picture

chipway commented
Title: All dependencies must be prefixed » All dependencies must be prefixed + capitalize name
Status: Needs work » Needs review
StatusFileSize
new Update_dependencies_format_in_info.yml_3015053-4.patch808 bytes

Thanks Aanal,
Here is the new patch. Please review.
I alsocapitalized a module name per coding standards.

Aanal.addweb’s picture

Aanal.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new dependency_Invite.png31.76 KB
new dependency_Invite2.png32.08 KB
new without_patch_Invite.png19.83 KB
new with_patch_Invite.png20.25 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.

renatog’s picture

renatog commented

Hi everyone. Good morning.

I applied the patch and Really works good!

+1 to it.

Thank you very much for your contribution.

Good Work.

Best,

  • RenatoG committed a1ced03 on 8.x-1.x authored by chipway 
    Issue #3015053 by chipway, Aanal.addweb, RenatoG: All dependencies must...
renatog’s picture

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

Status: Fixed » Closed (fixed)

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