-
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
Fix missing ValidatorMessage
stub in Python bindings
#65
Conversation
Codecov Report
@@ Coverage Diff @@
## main #65 +/- ##
=======================================
Coverage 19.50% 19.50%
=======================================
Files 17 17
Lines 3214 3214
=======================================
Hits 627 627
Misses 2587 2587 |
An API is changing, so should the Changelog be updated? |
CHANGELOG.md
Outdated
@@ -9,9 +9,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 | |||
### Fixed | |||
|
|||
- Fixed a typo in the Python type stubs for `ferveo.Keypair.secure_randomness_size()`. ([#61]) | |||
- Replaced raw tuples with `ValidatorMessage` in Python bindings. ([ferveo#131]) |
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.
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.
i.e. major version change...?
I think your changelog entries should be under 0.10.0
because of the breaking change.
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.
They should be under ### Changed
subsection, not ### Fixed
.
6c5f6cc
to
6bea0ba
Compare
Fixed & rebased |
Needs another rebase now that #63 has been merged. |
b1bf177
to
a077cd0
Compare
Rebased again. Added a small, unrelated change related to the previous PR that I didn't catch. |
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.
🎸
@piotr-roslaniec I thought we weren't merging with github dependencies?
|
@derekpierre, Yes, my bad, I will replace it with a new release nucypher/ferveo#132 shortly |
Type of PR:
Required reviews:
What this does:
ValidatorMessage
type in Python BindingsIssues fixed/closed:
AggregatedTranscript()
signature incorrect in.pyi
ferveo#129ferveo-pre-release@0.1.0-alpha.9
#62Why it's needed:
.pyi
stubs ferveo#131Notes for reviewers:
ferveo
versionferveo-pre-release@0.1.0-alpha.9
#62 since theferveo
version used is the child of the changes mentioned in the issue