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.cron
@@ -29,8 +29,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
@@ -50,7 +50,7 @@ The default values for the variables are set in [`defaults/main.yml`](https://gi
 ntp_state: started
 
 # The state of the NTP service at boot.
-ntp_enabled: yes
+ntp_enabled: true
 
 # A list of IP addresses to listen on.
 ntp_interfaces:

defaults/main.yml

@@ -5,7 +5,7 @@
 ntp_state: started
 
 # The state of the NTP service at boot.
-ntp_enabled: yes
+ntp_enabled: true
 
 # A list of IP addresses to listen on.
 ntp_interfaces:

meta/argument_specs.yml

@@ -16,7 +16,7 @@ argument_specs:
           - stopped
       ntp_enabled:
         type: "bool"
-        default: yes
+        default: true
         description: "The state of the NTP service at boot."
       ntp_interfaces:
         type: "list"
@@ -36,7 +36,7 @@ argument_specs:
       ntp_servers:
         type: "list"
         elements: str
-        required: no
+        required: false
         description: "A list of NTP servers and their options."
       ntp_timezone:
         type: "str"

molecule/default/converge.yml

@@ -1,8 +1,8 @@
 ---
 - name: Converge
   hosts: all
-  become: yes
-  gather_facts: yes
+  become: true
+  gather_facts: true
 
   roles:
     - role: robertdebock.cron

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

molecule/default/verify.yml

@@ -1,8 +1,8 @@
 ---
 - name: Verify
   hosts: all
-  become: yes
-  gather_facts: yes
+  become: true
+  gather_facts: true
 
   vars:
     _ntp_check_packages:
@@ -43,7 +43,7 @@
       register: npt_check_time_synchronised
       failed_when:
         - ntp_success_output not in npt_check_time_synchronised.stdout
-      changed_when: no
+      changed_when: false
       until:
         - npt_check_time_synchronised is succeeded
       retries: 16

tasks/assert.yml

@@ -5,29 +5,29 @@
     that:
       - ntp_state is defined
       - ntp_state in [ "started", "stopped" ]
-    quiet: yes
+    quiet: true
 
 - name: assert | Test ntp_enabled
   ansible.builtin.assert:
     that:
       - ntp_enabled is defined
       - ntp_enabled is boolean
-    quiet: yes
+    quiet: true
 
 - name: assert | Test ntp_interfaces
   ansible.builtin.assert:
     that:
       - ntp_interfaces is defined
       - ntp_interfaces is iterable
-    quiet: yes
+    quiet: true
 
 - name: assert | Test item in ntp_interfaces
   ansible.builtin.assert:
     that:
       - item.address is defined
       - item.address is string
       - item.address is not none
-    quiet: yes
+    quiet: true
   loop: "{{ ntp_interfaces }}"
   loop_control:
     label: "{{ item.address }}"
@@ -37,15 +37,15 @@
     that:
       - ntp_pool is defined
       - ntp_pool is iterable
-    quiet: yes
+    quiet: true
 
 - name: assert | Test item in ntp_pool
   ansible.builtin.assert:
     that:
       - item.name is defined
       - item.name is string
       - item.name is not none
-    quiet: yes
+    quiet: true
   loop: "{{ ntp_pool }}"
   loop_control:
     label: "{{ item.name }}"
@@ -54,7 +54,7 @@
   ansible.builtin.assert:
     that:
       - ntp_server is iterable
-    quiet: yes
+    quiet: true
   when:
     - ntp_server is defined
 
@@ -64,7 +64,7 @@
       - item.name is defined
       - item.name is string
       - item.name is not none
-    quiet: yes
+    quiet: true
   when:
     - ntp_server is defined
 
@@ -74,4 +74,4 @@
       - ntp_timezone is defined
       - ntp_timezone is string
       - ntp_timezone is not none
-    quiet: yes
+    quiet: true

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 packages

vars/main.yml

@@ -42,8 +42,8 @@ _ntp_timezone_packages:
 ntp_timezone_packages: "{{ _ntp_timezone_packages[ansible_os_family] | default(_ntp_timezone_packages['default']) }}"
 
 _ntp_timezone_supported:
-  default: no
-  Debian: yes
-  RedHat: yes
+  default: false
+  Debian: true
+  RedHat: true
 
 ntp_timezone_supported: "{{ _ntp_timezone_supported[ansible_os_family] | default(_ntp_timezone_supported['default']) }}"