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 Installation profiles to work well. So sooner is better.
Hope this helps.

CommentFileSizeAuthor
#3 dependency_Captcha Riddler2.png47.48 KBAanal.addweb
#3 dependency_Captcha Riddler.png31.71 KBAanal.addweb
#2 Update_dependencies_format_in_info.yml_3007012-2.patch372 byteschipway

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_3007012-2.patch372 bytes

Here is the patch. Please review.
I also
removed version keys because module is published on d.o.
removed the duplicate last empty line.

Aanal.addweb’s picture

Aanal.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new dependency_Captcha Riddler.png31.71 KB
new dependency_Captcha Riddler2.png47.48 KB

@chipway, Thanks for providing the patch. Your patch is working fine for me. As per the mentioned changes, I matched it with Pareview. PFA.

pifagor’s picture

pifagor
Location 🇺🇦 Rivne
commented

look good

voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented

Looks good

  • alex_optim committed 271337b on 8.x-1.x 
    Issue #3007012 by alex_optim, chipway: Update dependencies format in...
alex_optim’s picture

alex_optim
Location Tallinn
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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