From 9b377bbc1fc75f34a8cca9df667684256e1b6fc2 Mon Sep 17 00:00:00 2001
From: Ahmed Abdelhalim <pm@a14m.dev>
Date: Sat, 24 Jan 2026 18:14:48 +0100
Subject: Fix or ignore nvidia role linting issues

---
 roles/nvidia/tasks/install-archlinux.yml | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/roles/nvidia/tasks/install-archlinux.yml b/roles/nvidia/tasks/install-archlinux.yml
index cbcc2221..f11f71ba 100644
--- a/roles/nvidia/tasks/install-archlinux.yml
+++ b/roles/nvidia/tasks/install-archlinux.yml
@@ -23,15 +23,17 @@
 
 - name: "Get current mkinitcpio MODULES"
   become: true
-  ansible.builtin.shell:
-    cmd: "grep '^MODULES=' /etc/mkinitcpio.conf | sed 's/MODULES=(\\(.*\\))/\\1/'"
-  register: "current_modules"
+  ansible.builtin.shell: |
+    set -euo pipefail
+    grep '^MODULES=' /etc/mkinitcpio.conf | sed 's/MODULES=(\\(.*\\))/\\1/'
+  # noqa var-naming[no-role-prefix] as it describes the current kernel modules and not the nvidia modules
+  register: "current_kernel_modules"
   changed_when: false
 
 - name: "Add NVIDIA modules to mkinitcpio"
   become: true
   vars:
-    existing_modules: "{{ current_modules.stdout.split() | reject('in', nvidia_modules) | list }}"
+    existing_modules: "{{ current_kernel_modules.stdout.split() | reject('in', nvidia_modules) | list }}"
     all_modules: "{{ existing_modules + nvidia_modules }}"
   ansible.builtin.lineinfile:
     path: "/etc/mkinitcpio.conf"
-- 
cgit v1.2.3