ansistrano / deploy

Ansible role to deploy scripting applications like PHP, Python, Ruby, etc. in a capistrano style
https://ansistrano.com
MIT License
2.37k stars 343 forks source link

'include' directive is deprecated, replaced with 'include_tasks' #308

Closed sergids closed 6 years ago

ricardclau commented 6 years ago

👎 We are still supporting Ansible 2.3.

Ansible keeps breaking tons of things in every single release and not everybody can be bleeding edge in production

We will release a tag with the latest fixes this week and perhaps we can consider moving to Ansible 2.4 and merge this PR. I will leave the PR open until then

ricardclau commented 6 years ago

Please check the error messages, include_tasks is not allowed at the playbook level (this is test/main.yml)

Also, please squash your commits

ricardclau commented 6 years ago

Hi @sergids are you available in the next couple of days for fixing the things I pointed out?

Otherwise I will apply the changes myself in a fully working branch

ricardclau commented 6 years ago

Fixed in 6a7a825cc03bf27ab59dd3804ba561dbedb46650