Jose/ansible_role_proxmox_provision
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor ♻️: Refactor Proxmox repository management #5

Merged
Jose merged 5 commits from dev into main 2026-02-08 20:09:56 +01:00
Conversation 3 Commits 5 Files Changed 4 +4 -4
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit f96b3da66e - Show all commits

8
tasks/repos.yml
View File

@@ -2,11 +2,11 @@
- name: repos | Manage Proxmox repositories
vars:
repo_path:
- http://download\.proxmox\.com/debian/pve\
repo_path:
- http://download\.proxmox\.com/debian/pve\
block:
gitea-actions commented 2026-02-08 20:05:10 +01:00
Review
Copy Link

[Lines 6-8] [Score: 2] Const declarations are generally preferred for variable declaration. It makes the code more explicit and easier to understand, especially when dealing with complex data structures or multiple variables with the same type. Here, const repo_path could be used instead of var.

[Lines 6-8] [Score: 2] Const declarations are generally preferred for variable declaration. It makes the code more explicit and easier to understand, especially when dealing with complex data structures or multiple variables with the same type. Here, `const repo_path` could be used instead of `var`.
#Proxmox enterprise repo
# Proxmox enterprise repo
- name: repos | Check for Proxmox enterprise repo file
ansible.builtin.stat:
@@ -21,7 +21,7 @@
when: pve_enterprise_repo.stat.exists
register: enterprise_changed
#Proxmox no-subscription repo
# Proxmox no-subscription repo
- name: repos | Check for pve-install-repo.list
ansible.builtin.stat: