Apply new {project}:{module} format for dependencies in info.yml filehttps://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-about-your-module-with-an-infoyml-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 will be useful to install/update via composer. So sooner is better.



Hope this helps.

CommentFileSizeAuthor
#2 Update_dependencies_format_in_info.yml-3003252-1.patch458 bytesomrmankar
Screenshot_2018-09-29 https git drupal org project message_thread git 8 x-1 x.png24.46 KBomrmankar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

omrmankar created an issue. See original summary.

omrmankar’s picture

omrmankar
Primary language English
Location Mumbai, Maharashtra
CreditAttribution: omrmankar as a volunteer and commented
FileSize
Update_dependencies_format_in_info.yml-3003252-1.patch458 bytes

Here is the patch. Please review.

pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen at Randstad Digital for European Commission and European Union Institutions, Agencies and Bodies commented
Status: Needs work » Reviewed & tested by the community

Maybe the patch can be further improved by adding the missing newline at the end of the file, but that's minor.

This looks good, thanks @omrmankar!

NewZeal’s picture

NewZeal CreditAttribution: NewZeal as a volunteer commented

Commit added and ready to be pushed in the next few days.

  • New Zeal committed 82ec653 on 8.x-1.x authored by omrmankar 
    Issue #3003252 by omrmankar: Dependency namespacing in .info.yml
pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen at Randstad Digital for European Commission and European Union Institutions, Agencies and Bodies commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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