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.

Sign up for GitHub

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

Jump to bottom

Added flag to only generate the witness #39

Merged
merged 1 commit into from
Nov 30, 2023
Merged

Added flag to only generate the witness #39

merged 1 commit into from
Nov 30, 2023

Conversation

Nashtare
Copy link
Contributor

  • Faster but may get some tests to pass that shouldn't.

Reopening as it seems it mysteriously disappeared

@BGluth @Nashtare
Added flag to only generate the witness
81180f6 
- Faster but may get some tests to pass that shouldn't.
@Nashtare
Copy link
Contributor Author

I'll let you double-check before merging @BGluth

BGluth
BGluth approved these changes Nov 30, 2023
View reviewed changes
Copy link
Collaborator

@BGluth BGluth left a comment

Choose a reason for hiding this comment

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

Not sure what happened, but yeah LGTM.

@Nashtare Nashtare merged commit f0c3dbd into main Nov 30, 2023
2 checks passed
@Nashtare Nashtare deleted the witness_gen_only_flag branch February 26, 2024 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BGluth BGluth BGluth approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Nashtare @BGluth