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

Update polkadot-sdk to stable-2409 #566

Open
wants to merge 62 commits into
base: dev
Choose a base branch
from
Open

Update polkadot-sdk to stable-2409 #566

wants to merge 62 commits into from

Conversation

beqaabu
Copy link
Contributor

@beqaabu beqaabu commented Feb 6, 2025

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:

  • All GitHub Actions report PASS
  • Newly added code/functions have unit tests
    • Coverage tools report all newly added lines as covered
    • The positive scenario is exercised
    • Negative scenarios are exercised, e.g. assert on all possible errors
    • Assert on events triggered if applicable
    • Assert on changes made to storage if applicable
  • Modified behavior/functions - try to make sure above test items are covered
  • Integration tests are added if applicable/needed

Copy link

github-actions bot commented Feb 6, 2025

For full LLVM coverage report click here!

@beqaabu beqaabu force-pushed the forks-backup branch 6 times, most recently from 75954cb to a719873 Compare February 10, 2025 12:11
@beqaabu beqaabu force-pushed the forks-backup branch 4 times, most recently from ab1ea9d to a119957 Compare February 13, 2025 09:07
@beqaabu beqaabu marked this pull request as ready for review February 13, 2025 09:18
Copy link
Contributor

@atodorov atodorov left a comment

Choose a reason for hiding this comment

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

Small changes requested.

@beqaabu
Copy link
Contributor Author

beqaabu commented Feb 13, 2025

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

Copy link
Contributor

@atodorov atodorov left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@atodorov atodorov left a 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.

@beqaabu
Copy link
Contributor Author

beqaabu commented Feb 13, 2025

documentation nitpicks, can merge without them and follow up in another PR if you want.

I'll update this pr, would be cleaner

BradleyOlson64
BradleyOlson64 previously approved these changes Feb 13, 2025
@beqaabu beqaabu changed the title Forks backup Update polkadot-sdk to stable-2409 Feb 14, 2025
Copy link
Contributor

@DylanVerstraete DylanVerstraete left a comment

Choose a reason for hiding this comment

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

Good

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.

5 participants