Skip to content

upgrade to Polkadot v0.9.36#128

Merged
clangenb merged 4 commits into
integritee-network:masterfrom
litentry:polkadot-v0.9.36
Dec 22, 2022
Merged

upgrade to Polkadot v0.9.36#128
clangenb merged 4 commits into
integritee-network:masterfrom
litentry:polkadot-v0.9.36

Conversation

@Marky-Shi

@Marky-Shi Marky-Shi commented Dec 21, 2022

Copy link
Copy Markdown
Contributor

upgrade to polkadot-v0.9.36

@clangenb clangenb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks a lot, looks good to me!

@clangenb

Copy link
Copy Markdown
Contributor

We seem to have two issues here:

  1. We have a warning in the CI about deprecated use of a function.
  2. We will have to add another accepted licence in the about.toml: Apache-2.0 WITH LLVM-exception

@Marky-Shi

Copy link
Copy Markdown
Contributor Author

We seem to have two issues here:

  1. We have a warning in the CI about deprecated use of a function.
  2. We will have to add another accepted licence in the about.toml: Apache-2.0 WITH LLVM-exception

OK~ will fix these errors as soon as possible.

@Marky-Shi Marky-Shi requested a review from clangenb December 22, 2022 06:26

@clangenb clangenb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice, looks good to me! I opened an issue for assigning explicit call indexes to the remaining pallet calls: #129

@clangenb clangenb merged commit 1cdcfaa into integritee-network:master Dec 22, 2022
@Marky-Shi

Copy link
Copy Markdown
Contributor Author

Thanks!

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.

2 participants