Skip to content

Commit

Permalink
Merge pull request #242 from adamblake/compat-3.13
Browse files Browse the repository at this point in the history
Remove `pipes` for compatibility with Python 3.13
  • Loading branch information
minrk authored Nov 15, 2024
2 parents f86017e + 39c5ee5 commit 2883633
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
3 changes: 1 addition & 2 deletions chartpress.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@

import argparse
import os
import pipes
import re
import shlex
import shutil
Expand Down Expand Up @@ -68,7 +67,7 @@ def _log(message):
def _run_cmd(call, cmd, *, echo=True, **kwargs):
"""Run a command and echo it first with censoring of GITHUB_TOKEN."""
if echo:
cmd_string = " ".join(map(pipes.quote, cmd))
cmd_string = " ".join(map(shlex.quote, cmd))
github_token = os.getenv(GITHUB_TOKEN_KEY)
if github_token:
cmd_string = cmd_string.replace(github_token, "CENSORED_GITHUB_TOKEN")
Expand Down
2 changes: 0 additions & 2 deletions tests/test_repo_interactions.py
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
import contextlib
import os
import subprocess
import sys
import tempfile

import pytest
from conftest import cache_clear
Expand Down

0 comments on commit 2883633

Please sign in to comment.