vleeuwenmenno/dotfiles
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor formatting in update.py for improved readability
All checks were successful
Ansible Lint Check / check-ansible (push) Successful in 3s
Details
Nix Format Check / check-format (push) Successful in 1m13s
Details
Python Lint Check / check-python (push) Successful in 6s
Details

Browse Source
This commit is contained in:
Menno van Leeuwen
2025-09-24 18:40:25 +02:00
parent 78f3133a1d
commit e494369d11
1 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
bin/actions/update.py
View File

@@ -31,7 +31,9 @@ def help_message():
" --full-speed, -F Upgrade packages and use all available cores for compilation. (Default: 8 cores)", " --full-speed, -F Upgrade packages and use all available cores for compilation. (Default: 8 cores)",
) )
printfe("green", " --skip-check, -s Skip checking for dotfiles updates.") printfe("green", " --skip-check, -s Skip checking for dotfiles updates.")
printfe("green", " --system, -S Update system packages (flatpak, brew, apt, etc.)") printfe(
"green", " --system, -S Update system packages (flatpak, brew, apt, etc.)"
)
printfe("green", " --help, -h Display this help message.") printfe("green", " --help, -h Display this help message.")
return 0 return 0
@@ -246,10 +248,10 @@ def get_sudo_password_from_1password(username, hostname):
def get_distro(): def get_distro():
"""Detect the Linux distribution.""" """Detect the Linux distribution."""
try: try:
with open('/etc/os-release', 'r') as f: with open("/etc/os-release", "r") as f:
for line in f: for line in f:
if line.startswith('ID='): if line.startswith("ID="):
return line.split('=', 1)[1].strip().strip('"').lower() return line.split("=", 1)[1].strip().strip('"').lower()
except: except:
return None return None
@@ -296,7 +298,9 @@ def update_system_packages(sudo_password):
full_cmd = sudo_cmd + cmd full_cmd = sudo_cmd + cmd
printfe("cyan", f"Running: {' '.join(full_cmd)}") printfe("cyan", f"Running: {' '.join(full_cmd)}")
if sudo_password: if sudo_password:
result = subprocess.run(full_cmd, input=sudo_password + "\n", text=True, check=False) result = subprocess.run(
full_cmd, input=sudo_password + "\n", text=True, check=False
)
else: else:
result = subprocess.run(full_cmd, check=False) result = subprocess.run(full_cmd, check=False)
if result.returncode != 0: if result.returncode != 0: