Skip to content
This repository has been archived by the owner on Oct 26, 2022. It is now read-only.

Keep composer updated on boxes #69

Closed
wants to merge 2 commits into from
Closed

Conversation

becw
Copy link
Member

@becw becw commented Jan 14, 2020

Add a trigger to update the installed version of composer on vagrant up (#68).

This updates Composer itself, not the project dependencies, which should be generally good because Composer gets regular performance improvements and bug fixes.

In the infrequent cases where there's a breaking change in Composer (e.g. a schema change), it forces you to keep your project's composer.json up to date -- which is correct, because the general assumption is that any particular environment is running the latest composer.

@becw becw requested a review from byrond January 14, 2020 17:56
@becw
Copy link
Member Author

becw commented Jan 15, 2020

So the feedback on this that we discussed in slack was:

  • this is a good idea
  • but folks who weighed in would prefer that the Vagrantfile didn't run composer install -- it should just bring the machine up, and shouldn't touch your project

I'll make a new PR to reflect that.

@becw becw closed this Jan 15, 2020
@becw becw deleted the keep-composer-updated branch January 15, 2020 00:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant