-
Notifications
You must be signed in to change notification settings - Fork 2
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
Documentation restructure #105
Conversation
So everything works now. There is one small qol thing i am not sure is how pressing. The circom wit-calc does provide a warning of 35 unset signals.
it would be nice to figure out which ones these are and address this. I don't know how critical this is though. |
Latest constrains:
|
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. This looks like a pretty straightforward repo cleanup. Nice to see lines of code go down. I haven't combed over this PR line for line, but I did run tests, and it looks like we're passing. Nice work streamlining the repo Waylon, and good comments.
merge when you're rdy.
This PR currently does two things
Works on issue #100