[CI] Add checkmark emojis for passing builds (#170183)

This better matches the code formatter and I personally find the visual
indication valuable when I am scrolling/glancing at a comment.
This commit is contained in:
Aiden Grossman
2025-12-01 14:34:31 -08:00
committed by GitHub
parent 60513b8d6e
commit 28d2208f7d
2 changed files with 10 additions and 4 deletions

View File

@@ -184,8 +184,8 @@ def generate_report(
if return_code == 0:
report.extend(
[
"The build succeeded and no tests ran. This is expected in some "
"build configurations."
":white_check_mark: The build succeeded and no tests ran. "
"This is expected in some build configurations."
]
)
else:
@@ -272,6 +272,10 @@ def generate_report(
]
)
report.extend(_format_failures(ninja_failures, failure_explanations))
else:
report.extend(
["", ":white_check_mark: The build succeeded and all tests passed."]
)
if failures or return_code != 0:
report.extend(["", UNRELATED_FAILURES_STR])

View File

@@ -194,7 +194,7 @@ class TestReports(unittest.TestCase):
"""\
# Foo
The build succeeded and no tests ran. This is expected in some build configurations."""
:white_check_mark: The build succeeded and no tests ran. This is expected in some build configurations."""
),
)
@@ -308,7 +308,9 @@ class TestReports(unittest.TestCase):
"""\
# Foo
* 1 test passed"""
* 1 test passed
:white_check_mark: The build succeeded and all tests passed."""
)
),
)