Skip to content

Nip 13 POW#233

Merged
frnandu merged 4 commits intomasterfrom
nip13-pow
Sep 18, 2025
Merged

Nip 13 POW#233
frnandu merged 4 commits intomasterfrom
nip13-pow

Conversation

@nogringo
Copy link
Collaborator

Add the Nip13 class and pow methods for the events.

@codecov
Copy link

codecov bot commented Sep 13, 2025

Codecov Report

❌ Patch coverage is 89.13043% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.95%. Comparing base (056caf8) to head (0feefb5).
⚠️ Report is 14 commits behind head on master.

Files with missing lines Patch % Lines
...es/ndk/lib/domain_layer/entities/nip_01_event.dart 50.00% 4 Missing ⚠️
packages/ndk/lib/shared/nips/nip13/nip13.dart 97.36% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #233      +/-   ##
==========================================
+ Coverage   71.52%   71.95%   +0.42%     
==========================================
  Files         125      126       +1     
  Lines        4400     4446      +46     
==========================================
+ Hits         3147     3199      +52     
+ Misses       1253     1247       -6     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nogringo nogringo self-assigned this Sep 13, 2025
@nogringo nogringo requested review from 1-leo and frnandu September 13, 2025 12:08
Copy link
Contributor

@1-leo 1-leo left a comment

Choose a reason for hiding this comment

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

I thought about doing this with an event signer, but I like this event specific approach.
We may need to abstract it in the future to allow remote miners or mining in rust. However, this could be done without changing the current API.

@1-leo
Copy link
Contributor

1-leo commented Sep 17, 2025

Added a test checking validity; not sure why it fails. The test seems ok

@frnandu frnandu merged commit e200622 into master Sep 18, 2025
3 checks passed
@frnandu frnandu added this to the 0.6 milestone Sep 18, 2025
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.

3 participants