Skip to content

don't run tests with -v#2052

Merged
tgummerer merged 1 commit intomainfrom
tg/no-v
Mar 4, 2026
Merged

don't run tests with -v#2052
tgummerer merged 1 commit intomainfrom
tg/no-v

Conversation

@tgummerer
Copy link
Contributor

-v spams the CI output with much more information than is necessary, which makes it hard to figure out what actually failed (and makes the GitHub actions logs load extremely slowly). Remove it, as we should still get the full test output when the test fails, but much less noise for tests that succeed.

`-v` spams the CI output with much more information than is necessary,
which makes it hard to figure out what actually failed (and makes the
GitHub actions logs load extremely slowly).  Remove it, as we should
still get the full test output when the test fails, but much less
noise for tests that succeed.
@tgummerer tgummerer requested a review from a team as a code owner March 4, 2026 13:18
@iwahbe iwahbe added the impact/no-changelog-required This issue doesn't require a CHANGELOG update label Mar 4, 2026
@tgummerer tgummerer merged commit 52f563e into main Mar 4, 2026
20 checks passed
@tgummerer tgummerer deleted the tg/no-v branch March 4, 2026 15:37
tgummerer added a commit that referenced this pull request Mar 4, 2026
I missed that we don't use the makefile in CI in
#2052 🤦
tgummerer added a commit that referenced this pull request Mar 4, 2026
I missed that we don't use the makefile in CI in
#2052 🤦
@tgummerer tgummerer mentioned this pull request Mar 4, 2026
tgummerer added a commit that referenced this pull request Mar 5, 2026
I missed that we don't use the makefile in CI in
#2052 🤦
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact/no-changelog-required This issue doesn't require a CHANGELOG update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants