Skip to content

Install composer deps after beta deployment

Nick Sellen requested to merge composer-install-on-deploy into master

What does this MR do?

Currently we don't use composer for dependencies for the runtime app (only for development/testing). We had the first contribution that uses a composer dependency now, so we need to install composer dependencies when we deploy.

I didn't add any tests because it is a change to the deployment process, and that is not under test.

How confident are you it won't break things if deployed?

Not very confident. Composer install requires network access and filesystem access, should probably work, but better merged/watched by @NerdyProjects I think.

Links to related issues

!149 (merged) https://gitlab.com/foodsharing-dev/issues0/issues/190

Checklist

  • added a test (or explained why one is not needed)
  • no unrelated changes
  • asked someone for a code review

Merge request reports