-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update polkadot-sdk to stable-2409 #566
base: dev
Are you sure you want to change the base?
Conversation
For full LLVM coverage report click here! |
75954cb
to
a719873
Compare
ab1ea9d
to
a119957
Compare
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.
Small changes requested.
only the flaky smart contract tool test is failing https://github.com/gluwa/creditcoin3/actions/runs/13307767960/job/37163278848?pr=566. side note it is passing lcoally |
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.
lgtm
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.
documentation nitpicks, can merge without them and follow up in another PR if you want.
I'll update this pr, would be cleaner |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
b/c an actual iteration of this for loop may span between 2 consequtive blocks, or right around the edges of block time!
6c8a754
to
2c65250
Compare
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.
Good
Description of proposed changes
Updates polkadot-sdk dependencies to `stable-2409`, removes vendored tracing crates and uses moonbeam's tracing instead. Introduces a couple of breaking changes on the client's side, such as nodes that are using tracing features wont be able to use them unless they update their client to the latest version, after the runtime upgrade. full list of changes can be seen [here](https://gluwa.atlassian.net/wiki/spaces/CB/pages/1464074248/CC3+PolkadotSDK+and+Frontier+Upgrade)Practical tips for PR review & merge: