Skip to content

Fail GHA eval on non-empty stderr#381266

Merged
wolfgangwalther merged 2 commits intoNixOS:masterfrom
tweag:fail-eval-on-traces
Feb 14, 2025
Merged

Fail GHA eval on non-empty stderr#381266
wolfgangwalther merged 2 commits intoNixOS:masterfrom
tweag:fail-eval-on-traces

Conversation

@infinisil
Copy link
Member

Makes the GHA eval fail when stderr isn't empty, closing #371223.

Things done

  • Manually tested locally that warnings fail the eval.

This work is funded by Tweag and Antithesis

Add a 👍 reaction to pull requests you find important.

Previously stderr was mixed with the time stats

This allows checking stderr in the next commit
Just like the channel eval requires.
@infinisil infinisil marked this pull request as ready for review February 11, 2025 20:23
@github-actions github-actions bot added 6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions backport release-24.11 labels Feb 11, 2025
@github-actions github-actions bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Feb 11, 2025
@wolfgangwalther
Copy link
Contributor

I ran this 3 times:

  • once on master, it doesn't fail
  • once on haskell-updates, to my surprise it didn't fail either...
  • because the eval warnings I saw there yesterday got fixed in bb9e2af - so I ran it a 3rd time before that commit

As expected when those warnings show up, it errors out. The warnings are visible in the log output, too.

It does what it says!

@wolfgangwalther wolfgangwalther merged commit 1b2949a into NixOS:master Feb 14, 2025
39 checks passed
@nixpkgs-ci
Copy link
Contributor

nixpkgs-ci bot commented Feb 14, 2025

Successfully created backport PR for release-24.11:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants