Skip to content

Commit 38d62c4

Browse files
EliahKaganCopilotclaude
committed
Word shell mention of shlex.split more carefully
To avert the mistake of not removing `shell=True` when using it. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 010a7bb commit 38d62c4

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

git/cmd.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1222,8 +1222,8 @@ def execute(
12221222
12231223
On Unix-like systems, when migrating away from passing string commands with
12241224
``shell=True``, :func:`shlex.split` may serve as a transitional step in rare
1225-
cases, but this should be undertaken with extreme care. See the `command`
1226-
parameter above on the risks.
1225+
cases, with extreme care. (Drop ``shell=True`` and pass the resulting
1226+
sequence as the command.) See the `command` parameter above on the risks.
12271227
12281228
:param env:
12291229
A dictionary of environment variables to be passed to

0 commit comments

Comments
 (0)