ezsystems / launchpad

CLI tool to bootstrap an eZ Platform project Docker stack | #justcode
https://ezsystems.github.io/launchpad
MIT License
22 stars 27 forks source link

Add support for using composer install commands #58

Closed andrerom closed 5 years ago

andrerom commented 5 years ago
Q A
Branch? ?
Bug fix? no
New feature? yes
BC breaks? no
Fixed tickets N/A

As of v2.2 we started adding alias command in composer for install, it should be safe from one flavour to the next so I'm tempted to put it as a default but that would bump requirements to 2.2.

This is furthermore a requriments for adding Commerce support as it aliases several function calls: https://github.com/ezsystems/ezcommerce/blob/master/composer.json#L117-L123

Note1: Unsure how compatible the bash code added here is, might be better way to handle this also. Note2: Full Commerce support probably needs additional fixes on solr and potentially other things.

Plopix commented 5 years ago

thx @andrerom !

andrerom commented 5 years ago

Note, during execution scripts: in outputted in the output, so the pipe does not hide it all of what is performed in the if tests. I was thinking of echoing Run install before to make it seems on purpose, or checking with @vidarl how he would have done it ;)