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

Updates #86

Merged
merged 6 commits into from
Dec 11, 2023
Merged

Updates #86

merged 6 commits into from
Dec 11, 2023

Conversation

ftheirs
Copy link
Contributor

@ftheirs ftheirs commented Dec 7, 2023

Fix APDU documentation
Update SDK and dependencies
Make static analyzer happy :)

🔗 zboto Link

app/src/jubjub.c Fixed Show fixed Hide fixed
app/src/jubjub.c Fixed Show fixed Hide fixed
@ftheirs ftheirs force-pushed the dev branch 7 times, most recently from c78c528 to 33f1258 Compare December 8, 2023 18:44
app/src/sighash.c Dismissed Show dismissed Hide dismissed
@ftheirs ftheirs force-pushed the dev branch 2 times, most recently from 901035f to bc36323 Compare December 8, 2023 20:36
Copy link
Contributor

@carlosala carlosala left a comment

Choose a reason for hiding this comment

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

LGTM! Let's cherry-pick #87 before merging! 🚀

app/src/jubjub.c Outdated Show resolved Hide resolved
@ftheirs ftheirs force-pushed the dev branch 2 times, most recently from 921326d to ae9ce3a Compare December 11, 2023 13:56
Add description to install via the hub
@ftheirs ftheirs merged commit 75a5d21 into main Dec 11, 2023
39 checks passed
@ftheirs ftheirs deleted the dev branch December 11, 2023 14:13
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.

4 participants