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!

Comments

alonaoneill created an issue. See original summary.

alonaoneill’s picture

Assigned: alonaoneill » Unassigned
Status: Active » Needs review
StatusFileSize
new719 bytes

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

veerabhadra’s picture

@alonaoneill,
Patch does not apply.
error: patch failed: modules/jvector_demo/jvector_demo.info.yml:4
error: modules/jvector_demo/jvector_demo.info.yml: patch does not apply

veerabhadra’s picture

Status: Needs review » Needs work
veerabhadra’s picture

Status: Needs work » Needs review
StatusFileSize
new762 bytes

This patch fixes the issue.
Checking patch jvector.info.yml...
Checking patch modules/jvector_demo/jvector_demo.info.yml...
Applied patch jvector.info.yml cleanly.
Applied patch modules/jvector_demo/jvector_demo.info.yml cleanly.

alonaoneill’s picture

Status: Needs review » Needs work

My patch applied via STM. I don't see that problem.
Can you explain were you see it?