Pour tout problème contactez-nous par mail : support@froggit.fr | La FAQ :grey_question: | Rejoignez-nous sur le Chat :speech_balloon:

Skip to content
Snippets Groups Projects
Commit 3b5eb72d authored by Arthur BOUDREAULT's avatar Arthur BOUDREAULT
Browse files

:recycle:Refactor: Ansible-lint:

```
Include has some unintuitive behaviours depending on if it is running in a static or dynamic in play or in playbook context, in an effort to clarify behaviours we are moving to a new set modules (ansible.builtin.include_tasks, ansible.builtin.include_role, ansible.builtin.import_playbook, ansible.builtin.import_tasks) that have well established and clear behaviours.
This module will still be supported for some time but we are looking at deprecating it in the near future.
```

Fix: I have decided to go for ansible include_tasks module because it is more versatile and on a par with this module (for more info about differences between new modules, see [here](https://docs.ansible.com/ansible/latest/collections/ansible/builtin/include_module.html)
parent f6c6f1d1
No related branches found
No related tags found
1 merge request!7✅Resolve "ansible include module is to be deprecated"
This commit is part of merge request !7. Comments created here will be created in the context of that merge request.
......@@ -30,7 +30,7 @@
when: ynh_file_install.stat.exists == False
- name: Create extra domains
ansible.builtin.include: domains.yml
ansible.builtin.include_tasks: domains.yml
when: ynh_extra_domains
- name: Run first Yunohost diagnosis
......@@ -41,9 +41,9 @@
changed_when: False
- name: Add Yunohost users
ansible.builtin.include: users.yml
ansible.builtin.include_tasks: users.yml
when: ynh_users
- name: Install Yunohost apps
ansible.builtin.include: apps.yml
ansible.builtin.include_tasks: apps.yml
when: ynh_apps
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment