Dependency namespacing in .info.yml file

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 video_embed_youku-info-file-3013219-2.patch466 bytesvolkswagenchick

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Assigned: volkswagenchick » Unassigned
Status: Active » Needs review
StatusFileSize
new video_embed_youku-info-file-3013219-2.patch466 bytes

I uploaded a patch that aligns the dependency namespacing to Drupal standards. Thanks!

Status: Needs review » Needs work

The last submitted patch, 2: video_embed_youku-info-file-3013219-2.patch, failed testing. View results

alonaoneill’s picture

alonaoneill commented
Status: Needs work » Reviewed & tested by the community

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