Closed SebastianElvis closed 2 weeks ago
I am not sure whether this refactoring is neccessary to achieve #698 goal, but is nice to see less code in
app.go
so LGTM!
Not strictly necessary, but
All CreateUpgradeHandler
will have the same function signature
https://github.com/babylonchain/babylon/pull/703 gives an intuition on how is the AppKeepers
used
Closes #699 , step 1 of #698
This PR refactors the
BabylonApp
to move all keepers into a new struct, includingAppKeepers
that includes all keepers, underapp/keepers
.BabylonApp
comprisesAppKeepers
such that no interface is changed.AppKeepers.InitKeepers
function. The function is parameterised such that all config entries remain inapp/
folder.This is in line with Osmosis' approach, for formalising software upgrading and for modularity. See #698 for more details.