daodst / daodst/blockchain

logging

Open
#25 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Go
Stars
116
Forks
16
PR merge metrics
No merged PRs in 30d

Description

### Summary
Currently we use comet's interface for logging in the sdk. While this is an okay solution, zero log and other loggers have the opportunity to be typed. Secondly, the comet logger is limiting in what we would like to achieve with logging. For now, we have recreated the same interface to not cause large breaking changes, but once the sdk is migrated to the new interface and its loggers, we should break the interface, remove comet as a dependency and define the interface as we want.
### Work Breakdown

- [x] lntroduce logger interface same as comets
- [ ] Introduce noplogger and migrate the repo to use it.
- [ ] Introduce zero logger and migrate the repo to use it
- [ ] Add Filtering to zerolog
- [ ] Break the logger interface to remove comet dep and better fit our needs
- [x] Create a wrapper logger that can be passed to comet on start.

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.