daodst / daodst/blockchain

refactor: remove bytes/HexBytes

Open
#44 0 comments 0 reactions 1 assignee Claimed by @angstthrove View on GitHub
collections good first issue
Dominant language
Go
Stars
116
Forks
16
PR merge metrics
No merged PRs in 30d

Description

### Summary
Throughout the cosmos-sdk we have always used bytes.HexBytes from cmtbytes "github.com/cometbft/cometbft/libs/bytes". I have always questioned the need for this type even within tendermint. Its not an issue, but it would be nice to reduce the reliance on comet as a whole and remove this type if its not needed
### Problem Definition
Not a problem
### Proposal
Remove usage of bytes.HexBytes in order to reduce reliance of types from comet.

Who ever picks up this issue should first look if this change is breaking.

do we rely on the json encoding of the type or was this purely additive?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.