Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: nightly tests #1355

Merged
merged 2 commits into from
Oct 2, 2024
Merged

fix: nightly tests #1355

merged 2 commits into from
Oct 2, 2024

Conversation

DavePearce
Copy link
Collaborator

This makes the first step towards properly fixing the nightly tests.

@DavePearce DavePearce requested a review from gauravahuja October 2, 2024 13:48
@DavePearce DavePearce force-pushed the fix/issue-1353/fix-nightly-tests branch from f702261 to f998084 Compare October 2, 2024 13:52
This (hopefully) fixes the nightly tests by: (1) no longer supporting
`corset` for checking constraints (since this runs out of memory on
large traces); (2) uploads the test report after `go-corset` has run.
That will then allow us to see what has gone wrong.
This re-enables output to be written to the Github action console for
failing tests.  This has been previously disabled for reasons unknown,
but it ends up hiding important information.
@DavePearce DavePearce force-pushed the fix/issue-1353/fix-nightly-tests branch from f998084 to bfb1805 Compare October 2, 2024 14:03
@powerslider powerslider changed the title Fix/issue 1353/fix nightly tests fix: nightly tests Oct 2, 2024
@DavePearce DavePearce merged commit 3a1bfb0 into arith-dev Oct 2, 2024
5 checks passed
@DavePearce DavePearce deleted the fix/issue-1353/fix-nightly-tests branch October 2, 2024 14:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants