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

CommentFileSizeAuthor
#4 video-update-format-dependencies-3004694-4.patch303 bytesjqtnguyen
#2 video-update-format-dependencies-3004694-2.patch0 bytesjqtnguyen

Issue fork video-3004694

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

JQTNguyen created an issue. See original summary.

jqtnguyen’s picture

jqtnguyen commented
Status: Active » Needs review
StatusFileSize
new video-update-format-dependencies-3004694-2.patch0 bytes

Uploading patch for issue.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Status: Needs review » Needs work

Patch is empty. Marking needs work :)

jqtnguyen’s picture

jqtnguyen commented
Status: Needs work » Needs review
StatusFileSize
new video-update-format-dependencies-3004694-4.patch303 bytes

Let's run that back... Try this on for size!

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Status: Needs review » Reviewed & tested by the community

Looks good. Patch applied using simplytest.me

podarok made their first commit to this issue’s fork.

podarok opened merge request !5

  • podarok committed 1e2ba34 on 8.x-1.x 
    Issue #3004694 by JQTNguyen, volkswagenchick: Update Format of...
podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented
Status: Reviewed & tested by the community » Fixed

Thanks

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.