-
Notifications
You must be signed in to change notification settings - Fork 218
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
uint: bump version to 0.7 #136
Conversation
ethereum-types/Cargo.toml
Outdated
@@ -1,6 +1,6 @@ | |||
[package] | |||
name = "ethereum-types" | |||
version = "0.5.0" | |||
version = "0.5.1" |
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.
So the policy for dependent crates is to bump the the patch when updating a dependency?
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 don't think we have a hard policy on that but I think it's a good practise if applied sensibly? Do you see any downsides?
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 to me - was just curious what our convention was
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 think that depends on whether the change is breaking, I don't think it is for ethereum-types and primitive-types, but please correct me if I'm wrong.
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.
Or if you mean in general, I don't think we should do this in every case, but I'd like to publish new version of primitive-types as well.
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.
Yes of course my original comment is poorly phrased. Just when I've updated a dependency like this in the past (non breaking minor version) I was unsure about the correct bump for the parent crate.
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.
@sorpaas it seems like latest versions of ethereum-types
were published from primitives repo. Can you confirm if that's correct? What's the purpose of ethereum-types
in parity-common
if so?
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.
Can you also update the readme? It mentions v0.6
. Also perhaps add info about the new fuzzing tests?
to incorporate #126