FIX: Reorder yarn/bundle install for compatibility (#642) main
authorDavid Taylor <david@taylorhq.com>
Sat, 17 Sep 2022 12:22:34 +0000 (13:22 +0100)
committerGitHub <noreply@github.com>
Sat, 17 Sep 2022 12:22:34 +0000 (13:22 +0100)
commit8c40388ae4dabf1c0052ddd8eab4c72dbf7e0ca3
treed69a5eb30a97a6a19019b6042932f5fc2d93d6ce
parent2c907ea40382d7d9198e980368894221709faa25
FIX: Reorder yarn/bundle install for compatibility (#642)

Some existing templates (e.g. those written by following [the multisite guide](https://meta.discourse.org/t/multisite-configuration-with-docker/14084) use the `after_bundle_exec` hook to run application code like `multisite:migrate`. Relatively recently, the ruby app has started depending on `yarn` dependencies for basic functionality, and therefore requires `yarn install` to have been run to avoid errors.

This commit moves the `yarn install` step before `bundle install`, so that existing users of the `after_bundle_exec` hook do not need to update their templates.
templates/web.template.yml