Namespace Dependencies in .info.yml + misc coding standards [#3005133]

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.

Comment	File	Size	Author
#3	Opigno messaging.png	47.12 KB	zalak.addweb
#3	before_patch_Opigno messaging.png	28.49 KB	zalak.addweb
#3	after_patch_Opigno messaging.png	28.38 KB	zalak.addweb
#2	Update_dependencies_format_in_info.yml_3005133-2.patch	525 bytes	chipway

Comments

Comment #1

8 October 2018 at 14:58

chipway created an issue. See original summary.

Comment #2

chipway commented 8 October 2018 at 15:00

Assigned:	chipway	» Unassigned
Status:	Needs work	» Needs review

Status	File	Size
new	Update_dependencies_format_in_info.yml_3005133-2.patch	525 bytes

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

Comment #3

zalak.addweb commented 9 October 2018 at 19:48

Status:

Needs review

» Reviewed & tested by the community

Status	File	Size
new	after_patch_Opigno messaging.png	28.38 KB
new	before_patch_Opigno messaging.png	28.49 KB
new	Opigno messaging.png	47.12 KB

@chipway, Thanks for solving both the issues, it seems your patch is working fine for me. For Description related changes I checked your patch using simplytest.me & attached patch for the same & for Dependency related standard error I attached the Screenshot for pareview, PFA.