Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
While evaluating this project we noticed a coding standard that could be cleaned up in the info.yml file.
All dependencies must be prefixed by project name, So please apply new {project}:{module} format for dependencies in info.yml filehttps://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-...
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).
Thanks, patch to follow!
Comment | File | Size | Author |
---|---|---|---|
#2 | linky-infofile-3057020.patch | 387 bytes | volkswagenchick |
|
Comments
Comment #2
volkswagenchickHere's a patch that resolves the coding standards, thanks!
Comment #3
alonaoneill CreditAttribution: alonaoneill at Hook 42 commentedCode looks good. The dependency namespacing aligns with Drupal documentation standards.
Patch applied via simplytest.me
Marking as RTBC
Thank you for working on the module!
Comment #5
acbramley CreditAttribution: acbramley at PreviousNext commentedComment #6
volkswagenchickthanks