fixed lint

.ansible-lint

@@ -1,3 +1,4 @@
 skip_list:
   - '403' # allow for latest packages
   - name[casing]
+  - yaml[indentation]

.github/workflows/lint.yml

@@ -39,4 +39,3 @@ jobs:
       run: make deps
     - name: Format code
       run: make format
-

Makefile

@@ -36,7 +36,7 @@ lint-all:
 	make lint dir=playbooks
 
 lint:
-	ansible-lint $(dir) --exclude "roles/sprat.*" --exclude roles/docker_restore_container --exclude "roles/geerlingguy.*"
+	ansible-lint $(dir) --exclude "roles/sprat.*" --exclude roles/docker_restore_container --exclude "roles/geerlingguy.*" --exclude collections --exclude .github
 
 backup: deps
 	ansible-playbook playbooks/backup-docker-volumes.yml

playbooks/setup-homelab.yml

@@ -41,45 +41,45 @@
       ansible.builtin.include_vars: '../{{ vault_file }}'
       tags: [always]
   tasks:
-  - name: Install CIFS and other required packages for mounting with apt
-    apt:
-      name: "{{ item }}"
-      state: present
-    with_items:
-      - smbclient
-      - cifs-utils
+    - name: Install CIFS and other required packages for mounting with apt
+      ansible.builtin.apt:
+        name: "{{ item }}"
+        state: present
+      with_items:
+        - smbclient
+        - cifs-utils
 
-  - name: Check mountpoint exists.
-    ansible.builtin.file:
-      path: "/mnt/mergerfs"
-      state: directory
-      mode: '0777'
-      owner: root
-      group: root
+    - name: Check mountpoint exists.
+      ansible.builtin.file:
+        path: "/mnt/mergerfs"
+        state: directory
+        mode: '0777'
+        owner: root
+        group: root
 
-  - name: Check mountpoint exists.
-    ansible.builtin.file:
-      path: "/mnt/ssd0/downloads"
-      state: directory
-      mode: '0777'
-      owner: root
-      group: root
+    - name: Check mountpoint exists.
+      ansible.builtin.file:
+        path: "/mnt/ssd0/downloads"
+        state: directory
+        mode: '0777'
+        owner: root
+        group: root
 
-  - name: Mount shares.
-    mount:
-       state: "mounted"
-       fstype: "cifs"
-       path: /mnt/mergerfs
-       src: '//192.168.178.42/mergerfs'
-       opts: 'username=cianhatton,password={{ cianhatton_password }},dir_mode=0777,file_mode=0777,umask=0000'
+    - name: Mount shares.
+      mount:
+        state: "mounted"
+        fstype: "cifs"
+        path: /mnt/mergerfs
+        src: '//192.168.178.42/mergerfs'
+        opts: 'username=cianhatton,password={{ cianhatton_password }},dir_mode=0777,file_mode=0777,umask=0000'
 
-  - name: Mount shares.
-    mount:
-       state: "mounted"
-       fstype: "cifs"
-       path: /mnt/ssd0/downloads
-       src: '//192.168.178.42/downloads'
-       opts: 'username=cianhatton,password={{ cianhatton_password }},dir_mode=0777,file_mode=0777,umask=0000'
+    - name: Mount shares.
+      mount:
+        state: "mounted"
+        fstype: "cifs"
+        path: /mnt/ssd0/downloads
+        src: '//192.168.178.42/downloads'
+        opts: 'username=cianhatton,password={{ cianhatton_password }},dir_mode=0777,file_mode=0777,umask=0000'
 
 - name: Install Docker on Docker hosts.
   hosts: docker