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

Add a test for recursion #259

Merged
merged 4 commits into from Aug 30, 2022
Merged

Add a test for recursion #259

merged 4 commits into from Aug 30, 2022

Conversation

flaub
Copy link
Member

@flaub flaub commented Aug 28, 2022

The test only works with pure-prove and currently fails because it doesn't fit within MAX_CYCLES.

  • Adjust verify to work with no_std
  • Reduce verify dependencies
  • Add std to GuestOptions

@flaub flaub requested review from shkoo and jbruestle August 28, 2022 20:14
@flaub flaub self-assigned this Aug 28, 2022
Base automatically changed from flaub/check_code to main August 29, 2022 20:41
The test only works with `pure-prove` and currently fails because it doesn't fit within MAX_CYCLES.

* Adjust verify to work with `no_std`
* Reduce verify dependencies
* Add `std` to `GuestOptions`
@flaub flaub marked this pull request as ready for review August 29, 2022 22:08
@flaub flaub enabled auto-merge (squash) August 30, 2022 19:51
@flaub flaub merged commit f5836d7 into main Aug 30, 2022
@flaub flaub deleted the flaub/recursion branch August 30, 2022 20:01
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.

None yet

2 participants