Skip to content

Commit 8220d89

Browse files
Merge pull request ClickHouse#80241 from ClickHouse/ci_fix_job_commit_status
CI: Fix syntax error in post_commit_status
2 parents 31a334a + ddb1a48 commit 8220d89

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

ci/praktika/gh.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -146,6 +146,7 @@ def post_commit_status(cls, name, status, description, url):
146146
:param repo:
147147
:return: True or False in case of error
148148
"""
149+
description = description.replace("'", "'\"'\"'") # escape single quote
149150
status = cls.convert_to_gh_status(status)
150151
repo = _Environment.get().REPOSITORY
151152
command = (

ci/praktika/runner.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -510,7 +510,8 @@ def _post_run(
510510
description=result.info.splitlines()[0] if result.info else "",
511511
url=report_url,
512512
):
513-
print(f"ERROR: Failed to post failed commit status for the job")
513+
env.add_info("Failed to post GH commit status for the job")
514+
print(f"ERROR: Failed to post commit status for the job")
514515

515516
if workflow.enable_report:
516517
# to make it visible in GH Actions annotations

0 commit comments

Comments
 (0)