search

jdauphant / ansible-role-nginx

Ansible role to install and manage nginx configuration
655 stars 302 forks source link

Prefix keep_only_specified with nginx_ #212

Closed allaire closed 6 years ago

allaire commented 6 years ago

Just for uniformity purpose 👍

jdauphant commented 6 years ago

@allaire Thanks, this was bothering me too ;) You have to change it in the code also, the CI fail on that. https://github.com/jdauphant/ansible-role-nginx/blob/89ab5bd41c684bf304f45a1b4226152f13d8ea5c/tasks/main.yml#L25 https://github.com/jdauphant/ansible-role-nginx/blob/89ab5bd41c684bf304f45a1b4226152f13d8ea5c/tasks/main.yml#L29

allaire commented 6 years ago

@jdauphant I guess it was late, sorry about that! Fixed & rebased 👍

jdauphant commented 6 years ago

@allaire It's for that we have CI ;)