-
Notifications
You must be signed in to change notification settings - Fork 11
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
Release v0.14.0 #89
Release v0.14.0 #89
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #89 +/- ##
==========================================
- Coverage 23.46% 23.45% -0.01%
==========================================
Files 18 18
Lines 3520 3521 +1
==========================================
Hits 826 826
- Misses 2694 2695 +1 ☔ View full report in Codecov by Sentry. |
5bdb029
to
83a30c7
Compare
83a30c7
to
9729a42
Compare
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.
LGTM 👍
CHANGELOG.md
Outdated
|
||
### Changed | ||
|
||
- Fixed Python typings for `ThresholdMessageKit.decrypt_with_shared_secret()` and `combine_decryption_shares_simple()` ([#84]) |
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.
Is #84 a breaking change for "compatibility" if it is purely related to the binding signature, and not the underlying method? If it isn't a breaking change, could this instead be in a Fixed
block instead of in the Changed
and put below the Added
block?
Of course #83 is indeed a breaking change and should be left here, just channeling my inner Bogdan for changelogs format. 😄
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.
Indeed, it's a Fixed
change. Fixed.
ea1f35b
to
9fb1200
Compare
Type of PR:
Required reviews:
What this does:
Why it's needed:
taco-web
Notes for reviewers:
@nucypher/nucypher-core:0.14.1
, because I accidentally released WASM bundled with the old method instead of the new inlined method