Style changes

defaults/main.yml

@@ -1,2 +1,5 @@
 node_version: '12'
 node_gpg_key: 'https://deb.nodesource.com/gpgkey/nodesource.gpg.key'
+
+npm_packages: []
+npm_pre_packages: []

tasks/main.yml

@@ -11,48 +11,46 @@
 - name: update apt cache
   become: true
   apt:
-    update_cache: yes
+    update_cache: true
-  when: ansible_facts['os_family'] == "Debian" or ansible_facts['os_family'] == "Ubuntu"
+  when: ansible_facts['os_family'] == 'Debian' or ansible_facts['os_family'] == 'Ubuntu'
 
 - name: update pacman cache
   become: true
   pacman:
-    update_cache: yes
+    update_cache: true
-  when: ansible_facts['os_family'] == "Archlinux"
+  when: ansible_facts['os_family'] == 'Archlinux'
 
 - name: install needed tools
-  become: yes
+  become: true
   package:
-    name: '{{ item }}'
+    name: '{{ npm_pre_packages }}'
     state: present
-  loop: '{{ pre_packages }}'
 
 - name: add nodesource apt key
-  become: yes
+  become: true
   apt_key:
     url: '{{ node_gpg_key }}'
     state: present
-  when: ansible_facts['os_family'] == "Debian" or ansible_facts['os_family'] == "Ubuntu"
+  when: ansible_facts['os_family'] == 'Debian' or ansible_facts['os_family'] == 'Ubuntu'
 
 - name: add nodesource source
-  become: yes
+  become: true
   template:
     src: 'nodesource.list.j2'
     dest: '/etc/apt/sources.list.d/nodesource.list'
     mode: '0600'
-  when: ansible_facts['os_family'] == "Debian" or ansible_facts['os_family'] == "Ubuntu"
+  when: ansible_facts['os_family'] == 'Debian' or ansible_facts['os_family'] == 'Ubuntu'
 
 - name: create apt pinning file
-  become: yes
+  become: true
   template:
     src: 'nodesource.j2'
     dest: '/etc/apt/preferences.d/nodesource'
     mode: '0644'
-  when: ansible_facts['os_family'] == "Debian" or ansible_facts['os_family'] == "Ubuntu"
+  when: ansible_facts['os_family'] == 'Debian' or ansible_facts['os_family'] == 'Ubuntu'
 
 - name: install front-end packages
-  become: yes
+  become: true
   package:
-    name: '{{ item }}'
+    name: '{{ npm_packages }}'
     state: present
-  loop: '{{ packages }}'

vars/archlinux.yml

@@ -1,5 +1,3 @@
-pre_packages: []
+npm_packages:
-
-packages:
   - nodejs
   - npm

vars/debian-buster.yml

@@ -1,7 +1,7 @@
-pre_packages:
+npm_packages:
-  - gpg
-  - git
-
-packages:
   - nodejs
   - npm
+
+npm_pre_packages:
+  - gpg
+  - git

vars/ubuntu-focal.yml

@@ -1,8 +1,8 @@
-pre_packages:
+npm_packages:
+  - nodejs
+  - npm
+
+npm_pre_packages:
   - gpg
   - gpg-agent
   - openssh-client
-
-packages:
-  - nodejs
-  - npm