diff options
| author | Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> | 2024-02-08 19:30:51 +0000 |
|---|---|---|
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2024-02-08 19:30:51 +0000 |
| commit | 8691a385fbbfdde8bfc5b2aeb295280be2928778 (patch) | |
| tree | 82a9cff7d517fce9a147c602c6dfe2b9ff481af4 | |
| parent | 69b1991361f823e1638cf56a2f3c6d198d2b6631 (diff) | |
| parent | 78f6cedd5cf92dc20b47f3eae5f2b3c963655e0b (diff) | |
| download | external_updater-8691a385fbbfdde8bfc5b2aeb295280be2928778.tar.gz | |
Merge "Fix subprocess-run-check lint." into main am: 78f6cedd5c
Original change: https://android-review.googlesource.com/c/platform/tools/external_updater/+/2955629
Change-Id: I5052a9a32db7a9cebc9c447e3efc66cb682aaf3e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | git_utils.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/git_utils.py b/git_utils.py index 211ddd6..1a03088 100644 --- a/git_utils.py +++ b/git_utils.py @@ -244,7 +244,7 @@ def clean(proj_path: Path) -> None: def is_valid_url(proj_path: Path, url: str) -> bool: cmd = ['git', "ls-remote", url] - return subprocess.run(cmd, cwd=proj_path, stdin=subprocess.DEVNULL, + return subprocess.run(cmd, cwd=proj_path, check=False, stdin=subprocess.DEVNULL, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL, start_new_session=True).returncode == 0 |