According to Drupal standards, modules should include dependencies in the .info.yml file.
Dependencies should be namespaced in the format {project}:{module}, where {project} is the project name as it appears in the Drupal.org URL (e.g. drupal.org/project/views) and {module} is the module's machine name.
https://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-...

Patch to follow, thanks!

CommentFileSizeAuthor
#2 dependency_fix.patch468 bytesuditrawat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

uditrawat created an issue. See original summary.

uditrawat’s picture

uditrawat CreditAttribution: uditrawat commented
FileSize
dependency_fix.patch468 bytes

Please find attached patch

alonaoneill’s picture

alonaoneill CreditAttribution: alonaoneill at Hook 42 commented
Status: Active » Reviewed & tested by the community

Code looks good. The dependency namespacing aligns with Drupal documentation standards.
Patch applied via simplytest.me
Marking as RTBC
Thank you for working on the module!

  • aarnau committed d7a83cb on 8.x-1.x 
    Issue #3076532 by uditrawat: Dependency namespacing in .info.yml file
aarnau’s picture

aarnau CreditAttribution: aarnau at Hiberus commented
Status: Reviewed & tested by the community » Fixed

Thanks. Commited!

Status: Fixed » Closed (fixed)

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