Skip to content

Commit 0fa2e1a

Browse files
committed
chore: cleanup everything
1 parent 7ca90ba commit 0fa2e1a

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

.github/workflows/validate_modified_targets.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,13 +73,11 @@ jobs:
7373
id: prepare-summary
7474
run: |
7575
$(poetry env activate)
76-
python devel/summarize_site_validation.py validation_results.xml
7776
summary=$(
7877
python devel/summarize_site_validation.py validation_results.xml || echo "Failed to generate summary of test results"
7978
)
8079
deactivate
8180
echo "$summary" > validation_summary.md
82-
cat validation_summary.md
8381
8482
- name: Announce validation results
8583
if: steps.discover-modified.outputs.changed_targets != ''

devel/summarize_site_validation.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ def summarize_junit_xml(xml_path: Path) -> str:
1919

2020
summary_lines: list[str] = []
2121
summary_lines.append("#### Automatic validation of changes\n")
22-
summary_lines.append("| | F- Check | F+ Check |")
22+
summary_lines.append("| Target | F+ Check | F- Check |")
2323
summary_lines.append("|---|---|---|")
2424

2525
failures = int(suite.get('failures', 0))
@@ -45,7 +45,7 @@ def summarize_junit_xml(xml_path: Path) -> str:
4545
errors_detected = True
4646

4747
for result in results:
48-
summary_lines.append(f"| {result} | {results[result].get('F- Check', 'Error!')} | {results[result].get('F+ Check', 'Error!')} |")
48+
summary_lines.append(f"| {result} | {results[result].get('F+ Check', 'Error!')} | {results[result].get('F- Check', 'Error!')} |")
4949

5050
if failures > 0:
5151
summary_lines.append("\n___\n" +

0 commit comments

Comments
 (0)