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

feat: support crypto/ed25519.Verify #1863

Merged
merged 3 commits into from
Apr 17, 2024
Merged

Conversation

omarsy
Copy link
Member

@omarsy omarsy commented Mar 30, 2024

Support crypto/ed25519 on GnoVM

Contributors' checklist...
  • Added new tests, or not needed, or not feasible
  • Provided an example (e.g. screenshot) to aid review or the PR is self-explanatory
  • Updated the official documentation or not needed
  • No breaking changes were made, or a BREAKING CHANGE: xxx message was included in the description
  • Added references to related issues and PRs
  • Provided any useful hints for running manual tests
  • Added new benchmarks to generated graphs, if any. More info here.

@github-actions github-actions bot added the 📦 🤖 gnovm Issues or PRs gnovm related label Mar 30, 2024
@omarsy omarsy requested a review from a team as a code owner March 30, 2024 14:02
@omarsy omarsy changed the title feat: support crypto/ed25519 feat: support crypto/ed25519.Verify Mar 30, 2024
Copy link

codecov bot commented Apr 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.11%. Comparing base (3297d0c) to head (9fd7b67).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1863      +/-   ##
==========================================
- Coverage   47.77%   45.11%   -2.67%     
==========================================
  Files         393      464      +71     
  Lines       61630    68039    +6409     
==========================================
+ Hits        29443    30693    +1250     
- Misses      29717    34773    +5056     
- Partials     2470     2573     +103     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@thehowl
Copy link
Member

thehowl commented Apr 15, 2024

Thank you for the contribution! Sorry for the delay, I'll try to take a look in the next few days!

Copy link
Member

@zivkovicmilos zivkovicmilos left a comment

Choose a reason for hiding this comment

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

Looks good 💯

gnovm/stdlibs/crypto/ed25519/ed25519.go Show resolved Hide resolved
@omarsy omarsy requested review from a team, piux2 and moul as code owners April 16, 2024 17:47
@github-actions github-actions bot added 📦 🌐 tendermint v2 Issues or PRs tm2 related 📦 ⛰️ gno.land Issues or PRs gno.land package related labels Apr 16, 2024
@omarsy omarsy force-pushed the feat/add-ed25519 branch 2 times, most recently from d8e189c to eb77b4a Compare April 16, 2024 18:00
Co-authored-by: Miguel Victoria Villaquiran <Villaquiranm@users.noreply.github.com>
Copy link
Member

@thehowl thehowl left a comment

Choose a reason for hiding this comment

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

Pre-approving, good to merge after renaming

n0izn0iz and others added 2 commits April 17, 2024 12:09
Signed-off-by: Norman Meier <norman@samourai.coop>
@thehowl thehowl merged commit 75c9f9f into gnolang:master Apr 17, 2024
199 checks passed
@omarsy omarsy deleted the feat/add-ed25519 branch April 17, 2024 14:57
omarsy added a commit to TERITORI/gno that referenced this pull request Apr 21, 2024
Support crypto/ed25519 on GnoVM

<details><summary>Contributors' checklist...</summary>

- [x] Added new tests, or not needed, or not feasible
- [x] Provided an example (e.g. screenshot) to aid review or the PR is
self-explanatory
- [x] Updated the official documentation or not needed
- [ ] No breaking changes were made, or a `BREAKING CHANGE: xxx` message
was included in the description
- [ ] Added references to related issues and PRs
- [ ] Provided any useful hints for running manual tests
- [ ] Added new benchmarks to [generated
graphs](https://gnoland.github.io/benchmarks), if any. More info
[here](https://github.com/gnolang/gno/blob/master/.benchmarks/README.md).
</details>

---------

Signed-off-by: Norman Meier <norman@samourai.coop>
Co-authored-by: Miguel Victoria Villaquiran <Villaquiranm@users.noreply.github.com>
Co-authored-by: Norman Meier <norman@samourai.coop>
Co-authored-by: n0izn0iz <n0izn0iz@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 🌐 tendermint v2 Issues or PRs tm2 related 📦 ⛰️ gno.land Issues or PRs gno.land package related 📦 🤖 gnovm Issues or PRs gnovm related
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants