celestiaorg / celestiaorg/celestia-node
cmd: decouple binary and utils
Open
needs:triage
- Dominant language
- Go
- Stars
- 996
- Forks
- 1.1k
- Avg merge
- 1d 6h
- Merged PRs (30d)
- 34
Description
Suggested by @Wondertan:
"Decouple cmd/binary from cli/cmd utils. Move all the cmd/...go files to cli or lib/cli, then cmd would only be a place for our main apps/entrypoints.
Alternatively, we can move them to nodebuilder/node/cmd. I just looked at we have there and all the stuff there are node/admin related cmds - start, auth, utils for them, etc. So it logically makes sense to keep it there.
Generally, I like the idea of not keeping the cmd pkg with 0 code and only subpkgs with binaries, which is also idiomatic."
Contributor guide
Assessment
This issue has not been assessed yet.