andywimmer / ansible-grav

Ansible playbook to install NGINX, PHP7-FPM and Grav CMS
MIT License
13 stars 3 forks source link

Lots of modifications, feel free to accept them #1

Closed wfsaxton closed 7 years ago

wfsaxton commented 7 years ago

Not sure if you are accepting pull requests, but I took a look @ your work and figured I could build off of it for my own needs.

My own commits comments don't really do it justice because my first commit included a ton of changes. Here's the highlights:

andywimmer commented 7 years ago

Thanks for the pull request. I'm still learning Ansible obviously and your modifications help with that education so thank you!

I'll check out your changes locally and run some QA before merging this in. Cheers!

wfsaxton commented 7 years ago

Okay great. I'm new to Ansible as well and this has been a good learning experience.

I'm thinking of splitting this up into 2 different playbooks. One for the server provisioning, which only needs to be run each time you want a new server, and a second one for actual website deployment. Not sure what the "best practice" is for app deployment, but if that sounds useful let me know. Otherwise, I can just code my own thing.

andywimmer commented 7 years ago

Yeah this started out as just a way for me to learn Ansible with some tangible results; a working Grav installation as opposed to the default Nginx placeholder page. One thing I wanted to get sorted early was support for multiple distros and you have definitely set me down the right path there, so thank you.

I like to keep things simple and basic, so I think the goal of this repo will be basic server setup and Grav installation.

You are definitely welcome to 'code your own thing' though if you'd like to take things further!! 😃

wfsaxton commented 7 years ago

Okay, sounds good. I'm not sure how to "stop" adding my commits to this pull request, but I'm about to start making those changes for separate playbooks. Are you still considering this pull request or do you just want to use my fork as a reference and can pull in the stuff later?

andywimmer commented 7 years ago

I think I'm going to use your fork as reference and pull in changes as I have time to QA them.

wfsaxton commented 7 years ago

kk