Apply new {project}:{module} format for dependencies in info.yml

https://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-...

CommentFileSizeAuthor
#5 Add To Calendar.png27.82 KBriddhi.addweb
#4 Update_dependencies_format_in_info.yml-3000537-4.patch311 bytesvatsalkhanna
#2 Update_dependencies_format_in_info.yml-3000537-2.patch313 bytesvatsalkhanna

Comments

vatsalkhanna1992 created an issue. See original summary.

vatsalkhanna’s picture

vatsalkhanna commented
StatusFileSize
new Update_dependencies_format_in_info.yml-3000537-2.patch313 bytes

Please check the patch attached.

vatsalkhanna’s picture

vatsalkhanna commented
Status: Active » Needs review
vatsalkhanna’s picture

vatsalkhanna commented
StatusFileSize
new Update_dependencies_format_in_info.yml-3000537-4.patch311 bytes

Please check the updated patch, with project changed to "drupal".

riddhi.addweb’s picture

riddhi.addweb commented
StatusFileSize
new Add To Calendar.png27.82 KB

@vatsalkhanna1992, Thanks for the patch, your patch works as per the expected standards. I Review this issue with Pareview & Attached the screenshot for the same.

vatsalkhanna’s picture

vatsalkhanna commented

Thanks @riddhi.addweb, I am changing the status of this issue to RTC.

vatsalkhanna’s picture

vatsalkhanna commented
Status: Needs review » Reviewed & tested by the community
vatsalkhanna’s picture

vatsalkhanna commented
gg24’s picture

gg24 commented

Looks good to me. Thanks for the patch and review guys.

  • gg24 committed 765adc5 on 8.x-3.x authored by vatsalkhanna 
    Issue #3000537 by vatsalkhanna, riddhi.addweb, gg24: Apply new {project...
gg24’s picture

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

Status: Fixed » Closed (fixed)

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