From d1d3d08d70ac126c7a0c0e619398318e8e96fc51 Mon Sep 17 00:00:00 2001 From: Teal Dulcet Date: Fri, 22 Dec 2023 07:25:13 -0800 Subject: [PATCH] Fixed B006 (mutable-argument-default): Do not use mutable data structures for argument defaults --- management/utils.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/management/utils.py b/management/utils.py index a8fb5c2f..178a03bc 100644 --- a/management/utils.py +++ b/management/utils.py @@ -108,11 +108,13 @@ def sort_email_addresses(email_addresses, env): ret.extend(sorted(email_addresses)) # whatever is left return ret -def shell(method, cmd_args, env={}, capture_stderr=False, return_bytes=False, trap=False, input=None): +def shell(method, cmd_args, env=None, capture_stderr=False, return_bytes=False, trap=False, input=None): # A safe way to execute processes. # Some processes like apt-get require being given a sane PATH. import subprocess + if env is None: + env = {} env.update({ "PATH": "/sbin:/bin:/usr/sbin:/usr/bin" }) kwargs = { 'env': env,