fix linting and format

Author: danielefranceschi

.pre-commit-config.yaml

@@ -1,23 +1,23 @@
 repos:
-    - repo: https://github.com/pre-commit/pre-commit-hooks
-      rev: v4.0.1
-      hooks:
-        - id: check-yaml
-          args: [--allow-multiple-documents]
-        - id: end-of-file-fixer
-        - id: trailing-whitespace
-          args: [--markdown-linebreak-ext=md]
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.0.1
+    hooks:
+      - id: check-yaml
+        args: [--allow-multiple-documents]
+      - id: end-of-file-fixer
+      - id: trailing-whitespace
+        args: [--markdown-linebreak-ext=md]
 
-    - repo: https://github.com/adrienverge/yamllint
-      rev: v1.26.3
-      hooks:
-        - id: yamllint
-          args: [-c=.yamllint]
+  - repo: https://github.com/adrienverge/yamllint
+    rev: v1.26.3
+    hooks:
+      - id: yamllint
+        args: [-c=.yamllint]
 
-    - repo: https://github.com/robertdebock/pre-commit
-      rev: v1.2.3
-      hooks:
-        - id: ansible_role_find_unused_variable
-        - id: ansible_role_find_empty_files
-        - id: ansible_role_find_empty_directories
-        - id: ansible_role_fix_readability
+  - repo: https://github.com/robertdebock/pre-commit
+    rev: v1.2.3
+    hooks:
+      - id: ansible_role_find_unused_variable
+      - id: ansible_role_find_empty_files
+      - id: ansible_role_find_empty_directories
+      - id: ansible_role_fix_readability

defaults/main.yml

@@ -12,10 +12,10 @@ runner_version: "latest"
 runner_state: "started"
 
 # If found on the server, delete already existing runner service and install it again
-reinstall_runner: no
+reinstall_runner: false
 
 # Do not show Ansible logs which may contain sensitive data (registration token)
-hide_sensitive_logs: yes
+hide_sensitive_logs: true
 
 # GitHub address
 github_url: "https://github.com"
@@ -27,7 +27,7 @@ github_api_url: "https://api.github.com"
 access_token: "{{ lookup('env', 'PERSONAL_ACCESS_TOKEN') }}"
 
 # Is it the runner for organization or not?
-runner_org: no
+runner_org: false
 
 # Labels to apply to the runner
 runner_labels: []

tasks/collect_info.yml

@@ -2,76 +2,76 @@
 - name: Info collections
   check_mode: false
   block:
-  - name: Set complete API url for repo runner
-    ansible.builtin.set_fact:
-      github_full_api_url: "{{ github_api_url }}/repos/{{ github_owner | default(github_account) }}/{{ github_repo }}/actions/runners"
-    when: not runner_org and not github_enterprise
+    - name: Set complete API url for repo runner
+      ansible.builtin.set_fact:
+        github_full_api_url: "{{ github_api_url }}/repos/{{ github_owner | default(github_account) }}/{{ github_repo }}/actions/runners"
+      when: not runner_org and not github_enterprise
 
-  - name: Set complete API url for org runner
-    ansible.builtin.set_fact:
-      github_full_api_url: "{{ github_api_url }}/orgs/{{ github_owner | default(github_account) }}/actions/runners"
-    when: runner_org | bool and not github_enterprise
+    - name: Set complete API url for org runner
+      ansible.builtin.set_fact:
+        github_full_api_url: "{{ github_api_url }}/orgs/{{ github_owner | default(github_account) }}/actions/runners"
+      when: runner_org | bool and not github_enterprise
 
-  - name: Set complete API url for enterprise runner
-    ansible.builtin.set_fact:
-      github_full_api_url: "{{ github_api_url }}/enterprises/{{ github_enterprise }}/actions/runners"
-    when: github_enterprise
+    - name: Set complete API url for enterprise runner
+      ansible.builtin.set_fact:
+        github_full_api_url: "{{ github_api_url }}/enterprises/{{ github_enterprise }}/actions/runners"
+      when: github_enterprise
 
-  - name: Get registration token (RUN ONCE)
-    ansible.builtin.uri:
-      url: "{{ github_full_api_url }}/registration-token"
-      headers:
-        Authorization: "token {{ access_token }}"
-        Accept: "application/vnd.github.v3+json"
-      method: POST
-      status_code: 201
-      force_basic_auth: yes
-    register: registration
-    run_once: true
+    - name: Get registration token (RUN ONCE)
+      ansible.builtin.uri:
+        url: "{{ github_full_api_url }}/registration-token"
+        headers:
+          Authorization: "token {{ access_token }}"
+          Accept: "application/vnd.github.v3+json"
+        method: POST
+        status_code: 201
+        force_basic_auth: true
+      register: registration
+      run_once: true
 
-  - name: Check currently registered runners (RUN ONCE)
-    ansible.builtin.uri:
-      url: "{{ github_full_api_url }}"
-      headers:
-        Authorization: "token {{ access_token }}"
-        Accept: "application/vnd.github.v3+json"
-      method: GET
-      status_code: 200
-      force_basic_auth: yes
-    register: registered_runners
-    run_once: true
+    - name: Check currently registered runners (RUN ONCE)
+      ansible.builtin.uri:
+        url: "{{ github_full_api_url }}"
+        headers:
+          Authorization: "token {{ access_token }}"
+          Accept: "application/vnd.github.v3+json"
+        method: GET
+        status_code: 200
+        force_basic_auth: true
+      register: registered_runners
+      run_once: true
 
-  - name: Get Runner User IDs
-    ansible.builtin.command: id -u "{{ runner_user }}"
-    changed_when: false
-    register: runner_user_id
+    - name: Get Runner User IDs
+      ansible.builtin.command: id -u "{{ runner_user }}"
+      changed_when: false
+      register: runner_user_id
 
-  - name: Get Runner Group IDs
-    ansible.builtin.command: id -g "{{ runner_user }}"
-    changed_when: false
-    register: runner_user_group_id
+    - name: Get Runner Group IDs
+      ansible.builtin.command: id -g "{{ runner_user }}"
+      changed_when: false
+      register: runner_user_group_id
 
-  - name: Set runner_system variable
-    ansible.builtin.set_fact:
-      runner_system: "{{ 'osx' if ansible_system == 'Darwin' else 'linux' }}"
+    - name: Set runner_system variable
+      ansible.builtin.set_fact:
+        runner_system: "{{ 'osx' if ansible_system == 'Darwin' else 'linux' }}"
 
-  - name: Find the latest runner version (RUN ONCE)
-    ansible.builtin.uri:
-      url: "https://api.github.com/repos/{{ runner_download_repository }}/releases/latest"
-      headers:
-        Content-Type: "application/json"
-      method: GET
-      return_content: yes
-      status_code: 200
-      body_format: json
-    check_mode: false
-    register: api_response
-    run_once: true
-    become: false
-    delegate_to: localhost
-    when: runner_version == "latest"
+    - name: Find the latest runner version (RUN ONCE)
+      ansible.builtin.uri:
+        url: "https://api.github.com/repos/{{ runner_download_repository }}/releases/latest"
+        headers:
+          Content-Type: "application/json"
+        method: GET
+        return_content: true
+        status_code: 200
+        body_format: json
+      check_mode: false
+      register: api_response
+      run_once: true
+      become: false
+      delegate_to: localhost
+      when: runner_version == "latest"
 
-  - name: Get systemd service facts
-    ansible.builtin.service_facts:
-    register: service_facts
-    when: ansible_system == "Linux"
+    - name: Get systemd service facts
+      ansible.builtin.service_facts:
+      register: service_facts
+      when: ansible_system == "Linux"