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!: change to nums constants #111
feat!: change to nums constants #111
Conversation
Changed NUMS constants used for the generators to make use of domain separated labels and not use iterative hashing of the Ristretto base point.
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.
Seems fine.
Cargo.toml
Outdated
@@ -7,7 +7,7 @@ categories = ["cryptography"] | |||
homepage = "https://tari.com" | |||
readme = "README.md" | |||
license = "BSD-3-Clause" | |||
version = "0.13.2" | |||
version = "0.13.4" |
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.
Don't update the version, this is done in the tagging process
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.
The wrong version was my mistake, I'll update it with the next tag
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.
Ok cool
@@ -7,7 +7,7 @@ categories = ["cryptography"] | |||
homepage = "https://tari.com" | |||
readme = "README.md" | |||
license = "BSD-3-Clause" | |||
version = "0.13.4" | |||
version = "0.13.2" |
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.
breaking change should be 0.14
?
LGTM. Breaking changes need a bigger bump. Since we're pre 1.0, it should be to 0.14 |
Breaking change - any project based on a specific commitment in
tari_crypto
needs to be updated, as a commitment for the same value and blinding factor will now yield a different commitment. This was a necessary change as as described below.0.13.4
(this will align the version and tag as the previous version is0.13.2
and the previous tag is0.13.3
)