Apply new {project}:{module} format for dependencies in info.yml https://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-about-your-module-with-an-infoyml-file
It is supported since 8.0 and 7.40 Change Record: Project namespaces can now be added for module dependencies, and is now a Best Practice Define project dependencies in core module .info files).
It will be useful to use composer.
Hope this helps.

Comments

chipway created an issue. See original summary.

chipway’s picture

Assigned: chipway » Unassigned
Status: Needs work » Needs review
StatusFileSize
new1.41 KB

Here is a patch. Please review.

alonaoneill’s picture

Status: Needs review » Reviewed & tested by the community

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

itamair’s picture

Thanks @chipway so much. This has been deployed into last dev here: https://cgit.drupalcode.org/geocoder/commit/?id=c26f9c2918f90e44e5f62fdf...

Sorry I didn't see this before and couldn't grant you on this.

itamair’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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