imp: use storetypes instead of sdk types

[faddat]

Description

This PR changes the sdk.KVStore to storetypes.KVStore

Related PRs:

#3883

Commit Message / Changelog Entry

imp: use storetypes instead of sdk types

[faddat]

was definitely supposed to be in the 50 branch :)

[codecov-commenter]

Codecov Report

Merging #4009 (d435b68) into main (315680c) will not change coverage.
The diff coverage is 92.22%.

❗ Current head d435b68 differs from pull request most recent head c82599f. Consider uploading reports for the commit c82599f to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4009   +/-   ##
=======================================
  Coverage   78.88%   78.88%           
=======================================
  Files         187      187           
  Lines       12983    12983           
=======================================
  Hits        10241    10241           
  Misses       2313     2313           
  Partials      429      429           

Impacted Files	Coverage Δ
...odules/core/02-client/migrations/v7/solomachine.go	18.30% <0.00%> (ø)
...es/light-clients/06-solomachine/proposal_handle.go	69.56% <ø> (ø)
...les/light-clients/07-tendermint/proposal_handle.go	79.66% <ø> (ø)
modules/light-clients/07-tendermint/upgrade.go	73.86% <ø> (ø)
...dules/light-clients/06-solomachine/client_state.go	88.28% <75.00%> (ø)
modules/light-clients/09-localhost/client_state.go	91.66% <88.88%> (ø)
...27-interchain-accounts/controller/keeper/keeper.go	90.62% <100.00%> (ø)
...in-accounts/controller/migrations/v6/migrations.go	92.30% <100.00%> (ø)
.../apps/27-interchain-accounts/host/keeper/keeper.go	85.38% <100.00%> (ø)
modules/apps/29-fee/keeper/keeper.go	92.30% <100.00%> (ø)
... and 13 more

[damiannolan]

ACK pending CI, lgtm! 🙏

[DimitrisJim]

nice, thanks!

[chatton]

LGTM

[colin-axner]

woah, awesome this could be pulled out, thanks @faddat! Also thanks @damiannolan @chatton @DimitrisJim for the ultra fast reviews! ⚡ Y'all are the best!