Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

fix: Daily Blockchain Reference Test Workflow Failures #1588

Merged
merged 3 commits into from
Nov 29, 2024

Conversation

DavePearce
Copy link
Collaborator

Currently, workflow failures are being recorded as passing. The reason for this is simply a flag within the yaml file that was presumably copied over from the original gradle tests. This should now resolve the issus, and make sure that when the workflow fails --- we see that.

Currently, workflow failures are being recorded as passing.  The reason
for this is simply a flag within the yaml file that was presumably
copied over from the original gradle tests.  This should now resolve the
issus, and make sure that when the workflow fails --- we see that.
@DavePearce DavePearce linked an issue Nov 28, 2024 that may be closed by this pull request
@DavePearce DavePearce force-pushed the 1577-fix-daily-blockchain-reference-tests branch from 97b786c to 6bb1e8a Compare November 28, 2024 22:33
This puts through two minor fixes for the GeneralStateTests.  Firstly,
it adds notifications from the Rust corset side.  Secondly, it drops
parallelism from the go-corset side.
@DavePearce DavePearce force-pushed the 1577-fix-daily-blockchain-reference-tests branch 2 times, most recently from 03730a7 to 9fd3c1d Compare November 28, 2024 23:12
There appears to be an issue with running `gradle spotlessCheck` on the
CI server.
@DavePearce DavePearce force-pushed the 1577-fix-daily-blockchain-reference-tests branch from 9fd3c1d to 0ce8f15 Compare November 28, 2024 23:29
Copy link
Collaborator

@FlorianHuc FlorianHuc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG

@DavePearce DavePearce merged commit 02b33e0 into arith-dev Nov 29, 2024
5 checks passed
@DavePearce DavePearce deleted the 1577-fix-daily-blockchain-reference-tests branch November 29, 2024 07:13
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.

Fix Daily Blockchain Reference Tests
2 participants