Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@oech3
Copy link
Contributor

@oech3 oech3 commented Nov 13, 2025

We should catch error instead of ignoring.

@oech3
Copy link
Contributor Author

oech3 commented Nov 13, 2025

Very simple PR. But cc @naoNao89 for the sake.

@codspeed-hq
Copy link

codspeed-hq bot commented Nov 13, 2025

CodSpeed Performance Report

Merging #9256 will not alter performance

Comparing oech3:gtest-catch-err (e1548c4) with main (d432131)

Summary

✅ 123 untouched
⏩ 5 skipped1

Footnotes

  1. 5 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

@sylvestre sylvestre merged commit e2b5dd6 into uutils:main Nov 13, 2025
128 checks passed
@oech3 oech3 deleted the gtest-catch-err branch November 13, 2025 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants