pokt-network / pocket

Official implementation of the Pocket Network Protocol v1
https://pokt.network
MIT License
61 stars 33 forks source link

[Core] Use `cobra` for the Pocket node CLI #835

Open Olshansk opened 1 year ago

Olshansk commented 1 year ago

Objective

Use cobra in the node CLI binary consistency and simplicity of tooling across the board.

Origin Document

Screenshot 2023-06-16 at 11 41 55 AM Screenshot 2023-06-16 at 11 41 57 AM

Goals

Deliverable

Non-goals / Non-deliverables

General issue deliverables

Testing Methodology


Creator: @Olshansk Co-Owners: @bryanchriswhite


ChatGPT-ChatGPT (1)

Olshansk commented 1 year ago

@3scava1i3r If you're up for it, this is another big ticket item to hit in the future :)

3scava1i3r commented 1 year ago

Will look into it after the previous ones

bryanchriswhite commented 1 year ago

@Olshansk I :heart: the LLM CLI doc generation idea - that's a huge timesaver and a great starting point! :raised_hands:

I was talking with @h5law last week about some conventions that we may want to change regarding code organization in the CLI as right now there are some files with dozens of commands a thousand lines of code.

As peer discovery CLI work I've been doing is still very WIP, I'm not going to point to it as a good example just yet but I've been preferring to organize the CLI code in such a way that each subcommand: