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.66%. Comparing base (e857245) to head (90e1caf).

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   70.67%   68.66%   -2.01%     
==========================================
  Files         850      785      -65     
  Lines       55893    53536    -2357     
==========================================
- Hits        39502    36763    -2739     
- Misses      16391    16773     +382     
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 69 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