llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-infrastructure Author: Aiden Grossman (boomanaiden154) <details> <summary>Changes</summary> This enables reuse in other CI components, like premerge_advisor_explain.py. --- Full diff: https://github.com/llvm/llvm-project/pull/166604.diff 2 Files Affected: - (modified) .ci/generate_test_report_github.py (+3-12) - (modified) .ci/generate_test_report_lib.py (+11) ``````````diff diff --git a/.ci/generate_test_report_github.py b/.ci/generate_test_report_github.py index 08387de817467..18c5e078a5064 100644 --- a/.ci/generate_test_report_github.py +++ b/.ci/generate_test_report_github.py @@ -4,21 +4,10 @@ """Script to generate a build report for Github.""" import argparse -import platform import generate_test_report_lib -def compute_platform_title() -> str: - logo = ":window:" if platform.system() == "Windows" else ":penguin:" - # On Linux the machine value is x86_64 on Windows it is AMD64. - if platform.machine() == "x86_64" or platform.machine() == "AMD64": - arch = "x64" - else: - arch = platform.machine() - return f"{logo} {platform.system()} {arch} Test Results" - - if __name__ == "__main__": parser = argparse.ArgumentParser() parser.add_argument("return_code", help="The build's return code.", type=int) @@ -28,7 +17,9 @@ def compute_platform_title() -> str: args = parser.parse_args() report = generate_test_report_lib.generate_report_from_files( - compute_platform_title(), args.return_code, args.build_test_logs + generate_test_report_lib.compute_platform_title(), + args.return_code, + args.build_test_logs, ) print(report) diff --git a/.ci/generate_test_report_lib.py b/.ci/generate_test_report_lib.py index c9a2aaeb10f8c..48a6be903da41 100644 --- a/.ci/generate_test_report_lib.py +++ b/.ci/generate_test_report_lib.py @@ -4,6 +4,7 @@ """Library to parse JUnit XML files and return a markdown report.""" from typing import TypedDict +import platform from junitparser import JUnitXml, Failure @@ -305,3 +306,13 @@ def load_info_from_files(build_log_files): def generate_report_from_files(title, return_code, build_log_files): junit_objects, ninja_logs = load_info_from_files(build_log_files) return generate_report(title, return_code, junit_objects, ninja_logs) + + +def compute_platform_title() -> str: + logo = ":window:" if platform.system() == "Windows" else ":penguin:" + # On Linux the machine value is x86_64 on Windows it is AMD64. + if platform.machine() == "x86_64" or platform.machine() == "AMD64": + arch = "x64" + else: + arch = platform.machine() + return f"{logo} {platform.system()} {arch} Test Results" `````````` </details> https://github.com/llvm/llvm-project/pull/166604 _______________________________________________ llvm-branch-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
