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 concedra-info-3004750-2.patch804 bytesalonaoneill

Comments

alonaoneill created an issue. See original summary.

alonaoneill’s picture

alonaoneill commented
Assigned: alonaoneill » Unassigned
Status: Active » Needs review
StatusFileSize
new concedra-info-3004750-2.patch804 bytes

I uploaded a patch that corrects the dependency namespacing in the .info.yml file.
Thanks!

Willtg’s picture

Willtg commented
Status: Needs review » Reviewed & tested by the community

Code looks good. The dependency name-spacing aligns with Drupal documentation standards.
Marking as RTBC
Thank you for working on the module!

itothegore’s picture

itothegore commented

Thank you for working on and reviewed this patch!

itothegore’s picture

itothegore commented
Status: Reviewed & tested by the community » Fixed
itothegore’s picture

itothegore commented
Status: Fixed » Closed (fixed)