Skip to content

Commit ff2cc0e

Browse files
committed
ci: Don't print output twice in ci-util
Use `tee` rather than printing to both stdout and stderr.
1 parent aa25c33 commit ff2cc0e

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

.github/workflows/main.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ jobs:
3434
- name: Fetch pull request ref
3535
run: git fetch origin "$GITHUB_REF:$GITHUB_REF"
3636
if: github.event_name == 'pull_request'
37-
- run: python3 ci/ci-util.py generate-matrix >> "$GITHUB_OUTPUT"
37+
- run: set -e; python3 ci/ci-util.py generate-matrix | tee "$GITHUB_OUTPUT"
3838
id: script
3939

4040
test:

ci/ci-util.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -282,8 +282,6 @@ def emit_workflow_output(self):
282282
may_skip = str(self.may_skip_libm_ci()).lower()
283283
print(f"extensive_matrix={ext_matrix}")
284284
print(f"may_skip_libm_ci={may_skip}")
285-
eprint(f"extensive_matrix={ext_matrix}")
286-
eprint(f"may_skip_libm_ci={may_skip}")
287285
eprint(f"total extensive tests: {total_to_test}")
288286

289287
if error_on_many_tests and total_to_test > PrCfg.MANY_EXTENSIVE_THRESHOLD:

0 commit comments

Comments
 (0)