search

AdnanHodzic / containerized-wordpress-project

Automagically deploy & run WordPress with Let's Encrypt HTTPS encryption using Ansible & Docker
https://foolcontrol.org/?p=2002
GNU General Public License v3.0
42 stars 22 forks source link

Litte typo fixes #7

Closed iMartzen closed 4 years ago

iMartzen commented 4 years ago

Hello Adnan, Like I told in the other PR. I have made the roles compatible for RedHat. In this PR little typo fixes and also I think it is handy that the roles will install in the same directory. So people can look at the steps and maybe troubleshoot better if something is not going like expected. Hope to hear from you soon :) Best regards, Martzen

AdnanHodzic commented 4 years ago

Again, thank you, much appreciated.

Only one single remark, could you please leave out:

roles_path = ./roles

change as part of ansible.cfg file?

As this is good config if you want to have all roles locally, but I think in this case it's the best if all roles are simply downloaded and placed based on individual users Ansible configuration.

iMartzen commented 4 years ago

I left it out! Thanks for your quick response and probably it is best choice 👍