codec: Rename codec and marshaler interfaces

[robert-zaremba]

Renames. Trying to standarize the interfaces. See the issue for more details.

closes: #8413

---

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

• Targeted PR against correct branch (see CONTRIBUTING.md)
• Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
• Code follows the module structure standards.
• Wrote unit and integration tests
• Updated relevant documentation (docs/) or specification (x/<module>/spec/)
• Added relevant godoc comments.
• Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
• Re-reviewed Files changed in the Github PR explorer
• Review Codecov Report in the comment section below once CI passes

[codecov]

Codecov Report

Merging #9226 (8bc66b6) into master (c94e9eb) will not change coverage.
The diff coverage is 82.05%.

❗ Current head 8bc66b6 differs from pull request most recent head 061b173. Consider uploading reports for the commit 061b173 to get more accurate results

@@           Coverage Diff           @@
##           master    #9226   +/-   ##
=======================================
  Coverage   60.03%   60.03%           
=======================================
  Files         595      595           
  Lines       37353    37353           
=======================================
  Hits        22425    22425           
  Misses      12940    12940           
  Partials     1988     1988           

Impacted Files	Coverage Δ
client/context.go	54.86% <ø> (ø)
codec/yaml.go	50.00% <ø> (ø)
server/rosetta.go	0.00% <ø> (ø)
simapp/app.go	83.09% <ø> (ø)
simapp/genesis.go	100.00% <ø> (ø)
simapp/state.go	0.00% <ø> (ø)
simapp/utils.go	17.85% <ø> (ø)
testutil/network/network.go	90.00% <ø> (ø)
types/module/configurator.go	18.51% <ø> (ø)
types/module/module.go	68.36% <ø> (ø)
... and 96 more

[robert-zaremba]

I am not sure if this is relevant. I have found some instances of codec.Marshaler in md files and comments. Should you rename it to codec.Codec?

Thanks for checking, yes not all places in docs were updated. I've revisit it and updated.

[amaury1093]

lgtm, thanks!

[colin-axner]

Extra: I removed the BinaryBare part of the Marshal* methods / interface. It's really something Amino specific (no other codec has that name). Since, in SDK we never use Amino directly (only through LegacyAmino wrapper), we can easily update interfaces and remove it.
If folks don't like it, I can rollback that commit.

We happened to be using these functions in IBC. Is there a function I can migrate to that does the exact same functionality? We cannot change encoding values (ie they need to be exactly the same as before)

Is it safe to go from MustUnmarshalBinaryBare -> MustUnmarshal and the same for Marshal? I assume it should be fine since we are using proto?

Edit: I misread the CHANGELOG. I realize it is just the suffix that was removed (not the function itself) 👍

[xavierlepretre]

What is v in the comment?

[amaury1093]

yeah should be o. @xavierlepretre would you like to create a quick PR?