Change yes/no to true/false, adhere to yamlspec 1.2.2

.github/workflows/requirements2png.yml

@@ -31,5 +31,5 @@ jobs:
         uses: ad-m/github-push-action@master
         with:
           directory: ${{ github.repository }}
-          force: yes
+          force: true
           branch: png

.yamllint

@@ -11,9 +11,6 @@ rules:
   line-length: disable
   truthy:
     check-keys: no
-    allowed-values:
-      - "yes"
-      - "no"
 
 ignore: |
   .tox/

README.md

@@ -14,8 +14,8 @@ This example is taken from [`molecule/default/converge.yml`](https://github.com/
 ---
 - name: Converge
   hosts: all
-  become: yes
-  gather_facts: yes
+  become: true
+  gather_facts: true
 
   roles:
     - role: robertdebock.nextcloud
@@ -33,8 +33,8 @@ The machine needs to be prepared. In CI this is done using [`molecule/default/pr
 ---
 - name: Prepare
   hosts: all
-  become: yes
-  gather_facts: no
+  become: true
+  gather_facts: false
 
   roles:
     - role: robertdebock.bootstrap
@@ -53,8 +53,8 @@ The machine needs to be prepared. In CI this is done using [`molecule/default/pr
 
 - name: Continue prepare with facts
   hosts: all
-  become: yes
-  gather_facts: yes
+  become: true
+  gather_facts: true
 
   pre_tasks:
     - name: Include remi

handlers/main.yml

@@ -6,35 +6,35 @@
     cmd: php occ config:system:set memcache.distributed --value="\OC\Memcache\Redis"
   args:
     chdir: "{{ nextcloud_destination }}/nextcloud"
-  changed_when: yes
-  become: yes
+  changed_when: true
+  become: true
   become_user: "{{ nextcloud_httpd_owner }}"
 
 - name: Set redis host
   ansible.builtin.command:
     cmd: php occ config:system:set redis host --value="localhost"
   args:
     chdir: "{{ nextcloud_destination }}/nextcloud"
-  changed_when: yes
-  become: yes
+  changed_when: true
+  become: true
   become_user: "{{ nextcloud_httpd_owner }}"
 
 - name: Set redis port
   ansible.builtin.command:
     cmd: php occ config:system:set redis port --value="6379"
   args:
     chdir: "{{ nextcloud_destination }}/nextcloud"
-  changed_when: yes
-  become: yes
+  changed_when: true
+  become: true
   become_user: "{{ nextcloud_httpd_owner }}"
 
 - name: Set trusted domains
   ansible.builtin.command:
     cmd: php occ config:system:set trusted_domains 1 --value={{ nextcloud_domain_url }}
   args:
     chdir: "{{ nextcloud_destination }}/nextcloud"
-  changed_when: yes
-  become: yes
+  changed_when: true
+  become: true
   become_user: "{{ nextcloud_httpd_owner }}"
 
 - name: Set permissions
@@ -53,8 +53,8 @@
     cmd: php occ db:add-missing-indices
   args:
     chdir: "{{ nextcloud_destination }}/nextcloud"
-  changed_when: yes
-  become: yes
+  changed_when: true
+  become: true
   become_user: "{{ nextcloud_httpd_owner }}"
 
 - name: Restart httpd

molecule/default/converge.yml

@@ -1,8 +1,8 @@
 ---
 - name: Converge
   hosts: all
-  become: yes
-  gather_facts: yes
+  become: true
+  gather_facts: true
 
   roles:
     - role: ansible-role-nextcloud

molecule/default/prepare.yml

@@ -1,8 +1,8 @@
 ---
 - name: Prepare
   hosts: all
-  become: yes
-  gather_facts: no
+  become: true
+  gather_facts: false
 
   roles:
     - role: robertdebock.bootstrap
@@ -21,8 +21,8 @@
 
 - name: Continue prepare with facts
   hosts: all
-  become: yes
-  gather_facts: yes
+  become: true
+  gather_facts: true
 
   pre_tasks:
     - name: Include remi

molecule/default/verify.yml

@@ -1,8 +1,8 @@
 ---
 - name: Verify
   hosts: all
-  become: yes
-  gather_facts: yes
+  become: true
+  gather_facts: true
 
   vars_files:
     - ../../vars/main.yml
@@ -14,8 +14,8 @@
         cmd: php occ check
       args:
         chdir: "{{ nextcloud_destination }}/nextcloud"
-      changed_when: no
-      become: yes
+      changed_when: false
+      become: true
       become_user: "{{ nextcloud_httpd_owner }}"
 
     - name: Get status.php

tasks/assert.yml

@@ -6,69 +6,69 @@
       - nextcloud_version is defined
       - nextcloud_version is string
       - nextcloud_version is not none
-    quiet: yes
+    quiet: true
 
 - name: assert | Test nextcloud_domain_url
   ansible.builtin.assert:
     that:
       - nextcloud_domain_url is defined
       - nextcloud_domain_url is string
       - nextcloud_domain_url is not none
-    quiet: yes
+    quiet: true
 
 - name: assert | Test nextcloud_database_name
   ansible.builtin.assert:
     that:
       - nextcloud_database_name is defined
       - nextcloud_database_name is string
       - nextcloud_database_name is not none
-    quiet: yes
+    quiet: true
 
 - name: assert | Test nextcloud_database_user
   ansible.builtin.assert:
     that:
       - nextcloud_database_user is defined
       - nextcloud_database_user is string
       - nextcloud_database_user is not none
-    quiet: yes
+    quiet: true
 
 - name: assert | Test nextcloud_database_pass
   ansible.builtin.assert:
     that:
       - nextcloud_database_pass is defined
       - nextcloud_database_pass is string
       - nextcloud_database_pass is not none
-    quiet: yes
+    quiet: true
 
 - name: assert | Test nextcloud_database_host
   ansible.builtin.assert:
     that:
       - nextcloud_database_host is defined
       - nextcloud_database_host is string
       - nextcloud_database_host is not none
-    quiet: yes
+    quiet: true
 
 - name: assert | Test nextcloud_admin_user
   ansible.builtin.assert:
     that:
       - nextcloud_admin_user is defined
       - nextcloud_admin_user is string
       - nextcloud_admin_user is not none
-    quiet: yes
+    quiet: true
 
 - name: assert | Test nextcloud_admin_pass
   ansible.builtin.assert:
     that:
       - nextcloud_admin_pass is defined
       - nextcloud_admin_pass is string
       - nextcloud_admin_pass is not none
-    quiet: yes
+    quiet: true
 
 - name: assert | Test nextcloud_apps
   ansible.builtin.assert:
     that:
       - nextcloud_apps is iterable
-    quiet: yes
+    quiet: true
   when:
     - nextcloud_apps is defined
 
@@ -78,7 +78,7 @@
       - item.name is defined
       - item.name is string
       - item.name is not none
-    quiet: yes
+    quiet: true
   loop: "{{ nextcloud_apps }}"
   loop_control:
     label: "{{ item.name }}"
@@ -89,7 +89,7 @@
   ansible.builtin.assert:
     that:
       - nextcloud_settings is iterable
-    quiet: yes
+    quiet: true
   when:
     - nextcloud_settings is defined
 
@@ -103,7 +103,7 @@
       - item.section is string
       - item.section is not none
       - item.value is defined
-    quiet: yes
+    quiet: true
   loop: "{{ nextcloud_settings }}"
   loop_control:
     label: "{{ item.name }}"

tasks/main.yml

@@ -4,7 +4,7 @@
 - name: Import assert.yml
   ansible.builtin.import_tasks:
     file: assert.yml
-  run_once: yes
+  run_once: true
   delegate_to: localhost
 
 - name: Install requirements
@@ -21,8 +21,8 @@
 - name: Modify selinux settings
   ansible.posix.seboolean:
     name: "{{ item }}"
-    state: yes
-    persistent: yes
+    state: true
+    persistent: true
   when:
     - ansible_selinux.status is defined
     - ansible_selinux.status == "enabled"
@@ -32,7 +32,7 @@
   ansible.builtin.unarchive:
     src: "{{ nextcloud_archive }}"
     dest: "{{ nextcloud_destination }}"
-    remote_src: yes
+    remote_src: true
     owner: "{{ nextcloud_httpd_owner }}"
     group: "{{ nextcloud_httpd_group }}"
     creates: "{{ nextcloud_destination }}/nextcloud/occ"
@@ -43,8 +43,8 @@
     cmd: "{{ nextcloud_command }}"
     creates: "{{ nextcloud_destination }}/nextcloud/data/index.html"
     chdir: "{{ nextcloud_destination }}/nextcloud"
-  # no_log: yes
-  become: yes
+  # no_log: true
+  become: true
   become_user: "{{ nextcloud_httpd_owner }}"
   notify:
     - Set memcache.distributed
@@ -60,7 +60,7 @@
     cmd: php occ app:install {{ item.name }}
     chdir: "{{ nextcloud_destination }}/nextcloud"
     creates: "{{ nextcloud_destination }}/nextcloud/apps/{{ item.name }}"
-  become: yes
+  become: true
   become_user: "{{ nextcloud_httpd_owner }}"
   loop: "{{ nextcloud_apps }}"
   loop_control:

tasks/parameters.yml

@@ -7,8 +7,8 @@
         cmd: php occ config:app:get {{ item.section }} {{ item.name }} --output yaml
         chdir: "{{ nextcloud_destination }}/nextcloud"
       register: nextcloud_list_value
-      changed_when: no
-      become: yes
+      changed_when: false
+      become: true
       become_user: "{{ nextcloud_httpd_owner }}"
       failed_when:
         - nextcloud_list_value.stdout | from_yaml != item.value
@@ -17,8 +17,8 @@
       ansible.builtin.command:
         cmd: php occ config:app:set {{ item.section }} {{ item.name }} --value {{ item.value }}
         chdir: "{{ nextcloud_destination }}/nextcloud"
-      changed_when: no
-      become: yes
+      changed_when: false
+      become: true
       become_user: "{{ nextcloud_httpd_owner }}"
       notify:
         - Set permissions