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...

Comments

JQTNguyen created an issue. See original summary.

JQTNguyen’s picture

Assigned: JQTNguyen » Unassigned
Status: Active » Needs review
StatusFileSize
new380 bytes

Uploading patch for issue.

volkswagenchick’s picture

Status: Needs review » Reviewed & tested by the community

Code looks good, this patch addresses the dependency namespacing issue.
Patch applied via simplytest.me

Marking as RTBC

jacobbell84’s picture

Status: Reviewed & tested by the community » Closed (outdated)

Looks like this was taken care of by the module owner in #2957663: Upgrade to Conifguration Provider 2.x.