Skip to content

Commit d158f95

Browse files
author
Bryannah Hernandez
committed
check=true
1 parent 2affabb commit d158f95

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/sagemaker/serve/builder/requirements_manager.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ def detect_file_exists(self, dependencies: str = None) -> str:
3737
dependencies = self._capture_from_local_runtime()
3838

3939
# No additional dependencies specified
40-
if dependencies is None:
41-
return None
40+
# if dependencies is None:
41+
# return None
4242

4343
# Dependencies specified as either req.txt or conda_env.yml
4444
if dependencies.endswith(".txt"):
@@ -51,13 +51,13 @@ def detect_file_exists(self, dependencies: str = None) -> str:
5151
def _install_requirements_txt(self):
5252
"""Install requirements.txt file using pip"""
5353
logger.info("Running command to pip install")
54-
subprocess.run("pip install -r require.txt", shell=True)
54+
subprocess.run("pip install -r require.txt", shell=True, check=True)
5555
logger.info("Command ran successfully")
5656

5757
def _update_conda_env_in_path(self):
5858
"""Update conda env using conda yml file"""
5959
print("Updating conda env")
60-
subprocess.run("conda env update -f conda_in_process.yml", shell=True)
60+
subprocess.run("conda env update -f conda_in_process.yml", shell=True, check=True)
6161
print("Conda env updated successfully")
6262

6363
def _get_active_conda_env_name(self) -> str:

0 commit comments

Comments
 (0)