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 past migrations #2885

Merged
merged 10 commits into from
Jul 31, 2024
Merged

Remove past migrations #2885

merged 10 commits into from
Jul 31, 2024

Conversation

chexware
Copy link
Contributor

What does it do?

This PR removes migrations from previous releases.

What important points reviewers should know?

Is there something left for follow-up PRs?

What alternative implementations were considered?

Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?

What value does it bring to the blockchain users?

@noandrea noandrea added 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 labels Jul 29, 2024
@chexware chexware requested a review from RomarQ July 29, 2024 21:52
Copy link
Contributor

github-actions bot commented Jul 30, 2024

Coverage Report

@@                       Coverage Diff                        @@
##           master   aleks-remove-past-migrations      +/-   ##
================================================================
+ Coverage   80.84%                         80.97%   +0.13%     
- Files         282                            281       -1     
- Lines       81736                          81575     -161     
================================================================
- Hits        66079                          66054      -25     
- Misses      15657                          15521     -136     
Files Changed Coverage
/runtime/moonbeam/src/migrations.rs 100.00% (+86.42%) 🔼

Coverage generated Tue Jul 30 11:05:50 UTC 2024

@chexware chexware merged commit 258b136 into master Jul 31, 2024
37 checks passed
@chexware chexware deleted the aleks-remove-past-migrations branch July 31, 2024 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

4 participants