celestiaorg / celestiaorg/celestia-node

cmd: decouple binary and utils

Open
#2,754 1 comment 0 reactions 0 assignees View on GitHub
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.