All dependencies must be prefixed by project name, So please apply new {project}:{module} format for dependencies in info.yml 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 is useful for DrupalCi to download the right dependencies and Installation profiles to work well. So sooner is better.
Hope this helps.

Comments

chipway created an issue. See original summary.

chipway’s picture

Assigned: chipway » Unassigned
Status: Needs work » Needs review
StatusFileSize
new454 bytes

Here is the patch. Please review.
I also capitalized some module name per coding standards.

I didn't address CS fixed in https://www.drupal.org/project/advanced_shortcodes/issues/2888504

alonaoneill’s picture

Title: Must add prefix to dependencies » Must add prefix to dependencies and Capitalize the name of the module
Status: Needs review » Reviewed & tested by the community

Code looks good, the name of the module is now capitalized on the Extend page. "
The dependency namespacing aligns with Drupal documentation standards.
Marking as RTBC
Thank you for working on the module!

yang_yi_cn’s picture

Status: Reviewed & tested by the community » Fixed

fixed in 2.0.x branch.

yang_yi_cn’s picture

Status: Fixed » Closed (fixed)