Skip to content

Commit

Permalink
fix: Linting
Browse files Browse the repository at this point in the history
  • Loading branch information
tijmenstor committed Jul 17, 2024
1 parent 1a1b4b3 commit 4f08049
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/mpyl/steps/deploy/bpm/cluster.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ def deploy_diagram_to_cluster(logger: Logger, config: CamundaConfig):
environment_variables = {
"ZEEBE_ADDRESS": config.zeebe_credentials.cluster_id,
"ZEEBE_CLIENT_ID": config.zeebe_credentials.client_id,
"ZEEBE_CLIENT_SECRET": config.zeebe_credentials.client
"ZEEBE_CLIENT_SECRET": config.zeebe_credentials.client_secret,
}

custom_check_output(logger, command, environment_variables)
custom_check_output(logger, command, environment_variables=environment_variables)
8 changes: 6 additions & 2 deletions src/mpyl/utilities/subprocess/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ def custom_check_output(
command: Union[str, list[str]],
capture_stdout: bool = False,
use_print: bool = False,
environment_variables: dict[str, str] = None,
environment_variables: dict[str, str] = dict[None, None],
) -> Output:
"""
Wrapper around subprocess.Popen
Expand All @@ -38,7 +38,11 @@ def custom_check_output(
return Output(success=True, message=out)

with subprocess.Popen(
command, stdout=subprocess.PIPE, stderr=subprocess.STDOUT, text=True, env=environment_variables
command,
stdout=subprocess.PIPE,
stderr=subprocess.STDOUT,
text=True,
env=environment_variables,
) as process:
if not process.stdout:
raise RuntimeError(
Expand Down

0 comments on commit 4f08049

Please sign in to comment.