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

Bump version to 3.12.0 (to trigger update with the patched polkadot-sdk) #134

Merged
merged 3 commits into from
Dec 22, 2023

Conversation

nathanwhit
Copy link
Contributor

@nathanwhit nathanwhit commented Dec 20, 2023

Description of proposed changes

When merging #132 we didn't actually update the runtime version, so no runtime upgrade actually happened!


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

For full LLVM coverage report click here!

nathanwhit and others added 3 commits December 21, 2023 11:39
will require version bump whenever there's a change to any of the
dependencies which are consumed via GitHub instead of crates.io
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.

When merging #132 we didn't actually update the runtime version, so no runtime upgrade actually happened!

Rebased to dev and added a change to the version check script to catch the use case where dependencies consumed from GitHub have changed.

@atodorov atodorov merged commit af72066 into dev Dec 22, 2023
26 checks passed
@atodorov atodorov deleted the update-with-patch branch December 22, 2023 13:31
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