I believe it's best to include dependencies in a composer.json

CommentFileSizeAuthor
#3 commerce_license-require_interval_composer_json-3.patch352 bytesquiron
commerce_license-require_interval_composer_json.patch344 byteschrisrockwell

Comments

chrisrockwell created an issue. See original summary.

joachim’s picture

joachim commented
Status: Needs review » Needs work

It's Recurring Period we need now, rather than Interval.

quiron’s picture

quiron
Primary language Catalan
Location Barcelona
commented
StatusFileSize
new commerce_license-require_interval_composer_json-3.patch352 bytes

Then should it look like this new patch, right?

joachim’s picture

joachim commented

I thought d.org had some magic that kept the composer.json file in sync with the requirements in info.yml?

jsacksick’s picture

jsacksick commented
Status: Needs work » Closed (won't fix)

This isn't needed, as per https://www.drupal.org/docs/develop/using-composer/managing-dependencies....

Unless we require a minimum version, there isn't a need for that.