Andrew Berry
Andrew Berry
This should now be working in the latest addon release. Thanks!
👋 Is this an old project? `3.28.0` should be newer. If the project has drainpipe from before this PR in September, you may get errors on existing checkouts until things...
I was referring to the task version, not drainpipe. If you just updated to 3.12.1 now, that would explain the double-composer-install issue.
@beto-aveiga The composer checks package is now published to packagist. Let's pull that into Drainpipe as a new dev dependency in a new PR. Thanks!
I think this is fixed now, and we know that this PR doesn't work as is because it just breaks the cache on every execution. Closing!
@plopesc I think Terminus 4 requires PHP 8.2+, which means we'd need to drop support for that in Drainpipe? Or download the right version based on the current PHP version...
@davereid is this something you want to continue on, or document so someone else could pick it up?
Thanks for starting this. We probably want to be sure tests pass upstream before merging this!
What's interesting is the default config ddev generates has a node version in it. So I guess it would have been manually deleted? I think it would be fine for...
Why would you want to not specify the node version? At the least, I would have thought you'd want "auto" (which admittedly our Tugboat config won't do anything with). https://ddev.readthedocs.io/en/stable/users/configuration/config/#nodejs_version