Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Command Interface Spec #145

Closed
mappum opened this issue Oct 6, 2014 · 3 comments
Closed

Command Interface Spec #145

mappum opened this issue Oct 6, 2014 · 3 comments
Milestone

Comments

@mappum
Copy link
Contributor

mappum commented Oct 6, 2014

I've been thinking about how to implement a command interface to allow easily adding new functionality that will be accessible through CLI, HTTP RPC, etc. This stuff is important to get right since it's how we will create the interface developers will see when building stuff on top of IPFS, and good UX is a major factor for adoption.

Here's the design document where I've done some preliminary planning: https://gist.github.com/mappum/32cb25c9016ccd0219c8

Feel free to be critical, and write suggestions by forking the gist. If we can agree on a spec, I will quickly start hacking out the implementation.

RFC @jbenet @whyrusleeping @perfmode

@jbenet
Copy link
Member

jbenet commented Oct 7, 2014

So far thumbs up. addresses most of what i want out of it. Quick notes

  • request on HTTP transport should also be able to be done via a JSON POST body too (so both GET using query string params, and JSON POST body. libs should handle this for us)
  • docs in markdown are great for web (:+1:), though not sure how well they will turn out for commandline help/man pages. (Oh, now i see your Markdown -> ANSI comment, yeah!)
  • love the specifying of options nicely with a structure
  • probably mentioned it somewhere but subcommands in http can be /cmd/subcmd/subsubcmd

@whyrusleeping
Copy link
Member

How will you do subcommands via RPC?

@btc btc added this to the α milestone Oct 28, 2014
@btc
Copy link
Contributor

btc commented Nov 19, 2014

merged! 91fe0d9

@btc btc closed this as completed Nov 19, 2014
@aschmahmann aschmahmann mentioned this issue May 14, 2021
71 tasks
@aschmahmann aschmahmann mentioned this issue Dec 1, 2021
80 tasks
laurentsenta pushed a commit to laurentsenta/kubo that referenced this issue Feb 25, 2022
* enable pubsub and namesys pubsub to be enable via config
* rename Ipns key, switch type to Flag
* omit the fields if empty
laurentsenta pushed a commit to laurentsenta/kubo that referenced this issue Feb 25, 2022
* enable pubsub and namesys pubsub to be enable via config
* rename Ipns key, switch type to Flag
* omit the fields if empty
laurentsenta pushed a commit to laurentsenta/kubo that referenced this issue Mar 4, 2022
* enable pubsub and namesys pubsub to be enable via config
* rename Ipns key, switch type to Flag
* omit the fields if empty
laurentsenta pushed a commit to laurentsenta/kubo that referenced this issue Mar 4, 2022
* enable pubsub and namesys pubsub to be enable via config
* rename Ipns key, switch type to Flag
* omit the fields if empty
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants