Skip to content
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

Increase client version to v0.17.0 #1094

Merged
merged 4 commits into from
Dec 16, 2021
Merged

Increase client version to v0.17.0 #1094

merged 4 commits into from
Dec 16, 2021

Conversation

librelois
Copy link
Collaborator

What does it do?

What important points reviewers should know?

Is there something left for follow-up PRs?

What alternative implementations were considered?

Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?

What value does it bring to the blockchain users?

@librelois librelois added the B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes label Dec 16, 2021
Copy link
Contributor

@tgmichel tgmichel left a comment

Choose a reason for hiding this comment

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

Looks good to me, just a little change

@@ -1,5 +1,7 @@
# This file is automatically @generated by Cargo.
# It is not intended for manual editing.
version = 3
Copy link
Contributor

Choose a reason for hiding this comment

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

this should not be included I believe

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think it does, cargo adds it automatically, it's a new version of the Cargo.lock format that will have to be integrated anyway, might as well do it now

Copy link
Contributor

Choose a reason for hiding this comment

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

Previously we didn't want to include it but I'm good with including it if everyone agrees.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm surprised it is added now. Why, if it is automatic, wasn't it added before ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Because I updated my rust toolchain

@girazoki
Copy link
Collaborator

Looks good to me too

@librelois librelois added the A8-mergeoncegreen Pull request is reviewed well. label Dec 16, 2021
@librelois librelois merged commit 61aa903 into master Dec 16, 2021
@librelois librelois deleted the elois-client-v0.17.0 branch December 16, 2021 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A8-mergeoncegreen Pull request is reviewed well. B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants