diff --git a/.github/workflows/validate_modified_targets.yml b/.github/workflows/validate_modified_targets.yml index 99844216..a98483cc 100644 --- a/.github/workflows/validate_modified_targets.yml +++ b/.github/workflows/validate_modified_targets.yml @@ -73,13 +73,11 @@ jobs: id: prepare-summary run: | $(poetry env activate) - python devel/summarize_site_validation.py validation_results.xml summary=$( python devel/summarize_site_validation.py validation_results.xml || echo "Failed to generate summary of test results" ) deactivate echo "$summary" > validation_summary.md - cat validation_summary.md - name: Announce validation results if: steps.discover-modified.outputs.changed_targets != '' diff --git a/devel/summarize_site_validation.py b/devel/summarize_site_validation.py index 91a23e36..89d39750 100644 --- a/devel/summarize_site_validation.py +++ b/devel/summarize_site_validation.py @@ -19,7 +19,7 @@ def summarize_junit_xml(xml_path: Path) -> str: summary_lines: list[str] = [] summary_lines.append("#### Automatic validation of changes\n") - summary_lines.append("| | F- Check | F+ Check |") + summary_lines.append("| Target | F+ Check | F- Check |") summary_lines.append("|---|---|---|") failures = int(suite.get('failures', 0)) @@ -45,7 +45,7 @@ def summarize_junit_xml(xml_path: Path) -> str: errors_detected = True for result in results: - summary_lines.append(f"| {result} | {results[result].get('F- Check', 'Error!')} | {results[result].get('F+ Check', 'Error!')} |") + summary_lines.append(f"| {result} | {results[result].get('F+ Check', 'Error!')} | {results[result].get('F- Check', 'Error!')} |") if failures > 0: summary_lines.append("\n___\n" +