Please 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 is useful to install/update via composer. So sooner is better.

Also capitalize module name and remove unnecessary quotes following coding standards.

Hope this helps.

CommentFileSizeAuthor
#3 Screen Shot 2018-09-26 at 2.51.49 PM.png12.31 KBalonaoneill
#2 Update_dependencies_format_in_info.yml_3001112-2.patch1.14 KBchipway

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_3001112-2.patch1.14 KB

Here is the patch. Please review.

alonaoneill’s picture

alonaoneill commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Screen Shot 2018-09-26 at 2.51.49 PM.png12.31 KB

Code looks good, the name of the module is now capitalized on the Extend page. Screenshot provided."
The dependency namespacing aligns with Drupal documentation standards.
Marking as RTBC
Thank you for working on module!

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
commented
Status: Reviewed & tested by the community » Fixed
Related issues: +#3196906: D9 Validation + Config init failure

Solved in a different commit. Thanks for your contribution.

Status: Fixed » Closed (fixed)

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