Skip to content

Conversation

@aljo242
Copy link
Contributor

@aljo242 aljo242 commented Nov 7, 2025

Fully removes x/params. This has been deprecated since v0.47

additionally:

  • removes old non needed migrations
  • removes legacySubspace arguments
  • removes legacy query from gov

@codecov
Copy link

codecov bot commented Nov 7, 2025

Codecov Report

❌ Patch coverage is 68.51852% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.63%. Comparing base (aff4fe4) to head (c1556a1).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
contrib/x/crisis/module.go 0.00% 13 Missing ⚠️
contrib/x/crisis/keeper/migrations.go 0.00% 2 Missing ⚠️
x/genutil/client/cli/commands.go 0.00% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #25546      +/-   ##
==========================================
+ Coverage   68.39%   68.63%   +0.24%     
==========================================
  Files         856      785      -71     
  Lines       55773    53454    -2319     
==========================================
- Hits        38144    36687    -1457     
+ Misses      17629    16767     -862     
Files with missing lines Coverage Δ
simapp/app_di.go 78.89% <100.00%> (ø)
testutil/configurator/configurator.go 79.73% <ø> (-0.69%) ⬇️
testutil/network/network.go 77.37% <ø> (+0.17%) ⬆️
x/auth/keeper/migrations.go 100.00% <100.00%> (+93.10%) ⬆️
x/auth/module.go 75.00% <100.00%> (+2.16%) ⬆️
x/bank/module.go 66.32% <100.00%> (-0.03%) ⬇️
x/bank/simulation/genesis.go 92.59% <100.00%> (-7.41%) ⬇️
x/bank/types/params.go 88.88% <100.00%> (+1.93%) ⬆️
x/distribution/keeper/migrations.go 100.00% <100.00%> (+66.66%) ⬆️
x/distribution/module.go 70.45% <100.00%> (+0.56%) ⬆️
... and 13 more

... and 6 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants