-
Notifications
You must be signed in to change notification settings - Fork 160
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
Add cairo1-run pretty printing #1630
Conversation
Benchmark Results for unmodified programs 🚀
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1630 +/- ##
==========================================
- Coverage 97.67% 97.63% -0.04%
==========================================
Files 91 92 +1
Lines 37393 37464 +71
==========================================
+ Hits 36523 36579 +56
- Misses 870 885 +15 ☔ View full report in Codecov by Sentry. |
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.
Nice one Sir. John !
I think we just need to make some clippy fixies to handle an error and we are good to go
* Revert "Add cairo1-run pretty printing (#1630)" This reverts commit 061ba87. * First iteration progress * Copy tests from reverted PR * Fix expected test value * Format fixes + add continue * Fmt + better commentary * Update test file * Update commentary * Add test for reported bug * Update Changelog * Fix * Dereference references upon output serialization + Add test * Add comment to serialize_output function * Expand CHANGELOG description with new behaviour * fmt * Use doc commnt
Add cairo1-run pretty printing
Description
This PR adds pretty printing of the cairo1-run output for the following types:
Checklist