From 41a1de7bbbbe0aa851810fa140f6f77ce60b04e1 Mon Sep 17 00:00:00 2001 From: "bobo.yang" Date: Thu, 14 Nov 2024 12:39:29 +0800 Subject: [PATCH] refactor: Replace os.system with subprocess_run in git add - Replaced os.system with subprocess_run for git add command - Updated import to include subprocess_run from git_api - Ensured compatibility with existing git operations --- merico/github/commit/commit.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/merico/github/commit/commit.py b/merico/github/commit/commit.py index a305a3c..d2a8a6d 100644 --- a/merico/github/commit/commit.py +++ b/merico/github/commit/commit.py @@ -12,7 +12,7 @@ from lib.ide_service import IDEService sys.path.append(os.path.join(os.path.dirname(os.path.abspath(__file__)), "..")) from common_util import assert_exit # noqa: E402 -from git_api import get_issue_info, subprocess_check_output +from git_api import get_issue_info, subprocess_check_output, subprocess_run diff_too_large_message_en = ( "Commit failed. The modified content is too long " @@ -219,7 +219,7 @@ def rebuild_stage_list(user_files): subprocess_check_output(["git", "reset"]) # Stage all user_files for file in user_files: - os.system(f'git add "{file}"') + subprocess_run(["git", "add", file]) def get_diff():