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 h5p_challenge-info-3077727-2.patch290 bytesalonaoneill
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alonaoneill created an issue. See original summary.

alonaoneill’s picture

alonaoneill CreditAttribution: alonaoneill at Hook 42 commented
Assigned: alonaoneill » Unassigned
Status: Active » Needs review
FileSize
h5p_challenge-info-3077727-2.patch290 bytes
pjotr.savitski’s picture

pjotr.savitski CreditAttribution: pjotr.savitski commented
Assigned: Unassigned » pjotr.savitski
Status: Needs review » Active

Makes sense will push the change soon.

pjotr.savitski’s picture

pjotr.savitski CreditAttribution: pjotr.savitski commented

pjotr.savitski’s picture

pjotr.savitski CreditAttribution: pjotr.savitski commented
Status: Active » Fixed
pjotr.savitski’s picture

pjotr.savitski CreditAttribution: pjotr.savitski commented
Status: Fixed » Closed (fixed)