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

remove localAssets lazy migration #2797

Merged
merged 2 commits into from
May 20, 2024
Merged

Conversation

RomarQ
Copy link
Contributor

@RomarQ RomarQ commented May 8, 2024

What does it do?

Removes the lazy migration used for clearing local assets reserves, and also removes some unused types left behind.

@RomarQ RomarQ added B0-silent Changes should not be mentioned in any release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit labels May 8, 2024
Copy link
Contributor

github-actions bot commented May 8, 2024

Coverage Report

@@                         Coverage Diff                          @@
##           master   rq/remove-local-assets-migration      +/-   ##
====================================================================
+ Coverage   73.06%                             73.07%   +0.01%     
  Files         228                                228              
- Lines       69944                              69835     -109     
====================================================================
- Hits        51103                              51029      -74     
- Misses      18841                              18806      -35     
Files Changed Coverage
/pallets/moonbeam-lazy-migrations/src/lib.rs 87.50% (+29.17%) 🔼

Coverage generated Wed May 8 14:34:25 UTC 2024

@RomarQ RomarQ self-assigned this May 9, 2024
@RomarQ RomarQ added B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes not-breaking Does not need to be mentioned in breaking changes labels May 10, 2024
@RomarQ RomarQ merged commit b064f1a into master May 20, 2024
32 of 33 checks passed
@RomarQ RomarQ deleted the rq/remove-local-assets-migration branch May 20, 2024 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B0-silent Changes should not be mentioned in any release notes B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit not-breaking Does not need to be mentioned in breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants