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.We don't use composer_manager and have our own workflow for pulling in composer dependencies. Since the module doesn't really *require* composer_manager so much as *recommend* it, the dependency line should be removed.
Comments
Comment #2
twistor CreditAttribution: twistor as a volunteer commentedI agree. I know work is being done to make it irrelevant anyway.
May I inquire about what workflow you're using? Very curious to see how people are figuring out composer requirements.
I'm guessing leaving the requirement for 7.x is ok though I'm not entirely sure.
Comment #3
cweagansI think it's fine to require it in D7.
Right now, basically, we're just manually updating composer.json in DRUPAL_ROOT. We're in the process of moving to composer-merge-plugin, which will basically just include modules/*/*/composer.json (for instance, modules/custom/mymodule/composer.json). We're waiting for some of the composer stuff in core to shake out before we invest too much time in any solution, though.