-
Notifications
You must be signed in to change notification settings - Fork 206
Fix Fuzzer workflow #2232
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 Fuzzer workflow #2232
Conversation
|
|
Benchmark Results for unmodified programs 🚀
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2232 +/- ##
=======================================
Coverage 96.68% 96.68%
=======================================
Files 104 104
Lines 44027 44027
=======================================
Hits 42566 42566
Misses 1461 1461 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| schedule: | ||
| # At the end of every day | ||
| - cron: "0 0 * * *" | ||
| workflow_dispatch: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added this so we can run it manually if we ever find ourselves trying to debug it again and don't have to wait for the specific time it runs
| arbitrary = { version = "1.3.0", features = ["derive"] } | ||
| cairo-vm = { path = "../vm", features = ["test_utils"] } | ||
| honggfuzz = "0.5.55" | ||
| honggfuzz = "=0.5.58" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the CI we are using cargo hfuzz 0.5.58 so it has to match with honggfuzz
JulianGCalderon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Fix Fuzzer workflow
Closes #2228
Checklist