[WIP] add initial vagrant configuration#1956
[WIP] add initial vagrant configuration#1956maglnet wants to merge 22 commits intoRunalyze:masterfrom
Conversation
laufhannes
left a comment
There was a problem hiding this comment.
I personally don't know anything about Vagrant, so I can't provide profound feedback and I won't be able to provide support for this part of the repo later on, but I guess that's not a problem at all.
Concerning the file locations: Are the files in data/vagrant/ meant to be edited by users? The data/ directory is meant for all installation-specific files that won't be touched by updates, so that directory should be empty. I guess app/vagrant/ fits better, but I don't know if there's any best practice.
@mipapo does probably know more about this topic than I do.
|
Just a short explanation about vagrant: The files within Am 3. Oktober 2016 21:43:51 MESZ, schrieb Hannes notifications@github.com:
|
7bf6c36 to
4c24af0
Compare
|
btw: I have no idea how to fix the travis build error, seems this has something to do with automatic deployment: |
|
I think we can ignore travis for this pr, as there are no changes in any php file. Travis fails because of https://docs.travis-ci.com/user/pull-requests/#Pull-Requests-and-Security-Restrictions. I guess we need to wrap the first four lines of |
update composer if necessary add Runalyze 4.x dependencies
…s installed before
b2213dd to
2c7be57
Compare
This should make it easier for developers to setup a working Runalyze installation.
If you have vagrant installed,
vagrant upshould setup a virtual machine, which has runalyze installed and is available at http://localhost:8080Todo:
composer installif novendor/directory existsconfig.ymlif it doesn't existfeel free to give your feedback :)