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

feat: upgrade to polkadot v0.9.27 #394

Merged
merged 16 commits into from
Aug 10, 2022
Merged

feat: upgrade to polkadot v0.9.27 #394

merged 16 commits into from
Aug 10, 2022

Conversation

wischli
Copy link
Contributor

@wischli wischli commented Aug 5, 2022

fixes KILTProtocol/ticket#2130

  • upgrades to latest Polkadot v0.9.27
  • bumps rust nightly to latest version supported by Polkadot rustc 1.64.0-nightly (7fe022f5a 2022-07-24) from 2022-05-30
  • ci: bumps sr-tool to 1.62.0 from 1.60.0
  • fixes a bunch of new clippy complaints (mainly requiring Eq to be derived when PartialEq is implemented)
  • fixes clippy issues for clone runtime
  • removes license check in rustfmt as it is deprecated now (see Planned removal of license_template_path rust-lang/rustfmt#5103 )

Note worthy Substrate PRs

Migrations

  • No migrations included or required

TODO

  • Investigate new solution for deprecated license checking in rustfmt
  • Fix clippy

Checklist:

  • I have verified that the code works
    • No panics! (checked arithmetic ops, no indexing array[3] use get(3), ...)
  • I have verified that the code is easy to understand
    • If not, I have left a well-balanced amount of inline comments
  • I have left the code in a better state
  • I have documented the changes (where applicable)

@wischli wischli marked this pull request as ready for review August 8, 2022 12:08
@wischli wischli self-assigned this Aug 8, 2022
Copy link
Member

@ntn-x2 ntn-x2 left a comment

Choose a reason for hiding this comment

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

Very nice cleanups! I just have a couple of questions.

.gitlab-ci.yml Outdated Show resolved Hide resolved

# KILT pallets & primitives
kilt-support = {path = "../../support", default-features = false, optional = true}
Copy link
Member

Choose a reason for hiding this comment

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

Was this failing? Or why did you add it in?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was a clippy complaint

Copy link
Member

Choose a reason for hiding this comment

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

Interesting. Mine does not, plus I do not see it being used anywhere inside that crate. I see that its usage has been commented out as part of the // TODO: Enable after adding delegation pallet, so that is probably the reason. If we don't plan to uncomment those, then perhaps we can remove it from the Cargo file as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You are right, I suppose I added these comments after I added it to the dependencies. Fixed in 75c4255

@wischli wischli enabled auto-merge (squash) August 10, 2022 08:05
@wischli wischli merged commit 31cb03d into develop Aug 10, 2022
@wischli wischli deleted the wf/2130-polkadot-v0.9.27 branch August 10, 2022 08:18
wischli added a commit that referenced this pull request Aug 30, 2022
* feat: upgrade to Polkadot v0.9.27

* chore: bump nightly, sr-tool, base docker img

* fix: clippy clone runtime

* fix: clippy

* fix: remove deprecated license fmt checker

rust-lang/rustfmt#5103

* fix: clippy 2

* fix: clone

* chore: bump sub deps

* fix: clippy

* chore: bump serde

* ci: fix bump srtool to 1.62.0

* fix: remove kilt-support dep from clone runtime

* ci: bump parity base img
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants