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

Add config flag to cmd line interface #86

Merged
merged 2 commits into from
Sep 17, 2014
Merged

Conversation

whyrusleeping
Copy link
Member

No description provided.

@whyrusleeping whyrusleeping added the status/in-progress In progress label Sep 17, 2014
@jbenet
Copy link
Member

jbenet commented Sep 17, 2014

Made some comments, otherwise LGTM.

@jbenet
Copy link
Member

jbenet commented Sep 17, 2014

though one meta comment is that maybe instead of specifying the config file, we should be specifying the ipfs root (~/.go-ipfs) and assume $ROOT/config. This guides users to consistent ipfs repositories.

Though this has different semantics, so idk. Advantage of config file flag is that potentially there are installations that require nothing but the config file (i.e. don't need a whole directory).

I think: let's merge this in and keep the above ^ in consideration for the future.

@whyrusleeping whyrusleeping merged commit 2607bee into master Sep 17, 2014
@whyrusleeping whyrusleeping removed the status/in-progress In progress label Sep 17, 2014
@jbenet jbenet deleted the feat/config-flag branch September 20, 2014 11:41
ribasushi pushed a commit that referenced this pull request Jul 4, 2021
fix: correctly add the skip option to Warningf
@aschmahmann aschmahmann mentioned this pull request Dec 1, 2021
80 tasks
laurentsenta pushed a commit to laurentsenta/kubo that referenced this pull request Feb 25, 2022
…/multiformats/go-multiaddr-0.2.2

chore(deps): bump github.com/multiformats/go-multiaddr from 0.2.1 to 0.2.2
laurentsenta pushed a commit to laurentsenta/kubo that referenced this pull request Feb 25, 2022
…dules/github.com/multiformats/go-multiaddr-0.2.2

chore(deps): bump github.com/multiformats/go-multiaddr from 0.2.1 to 0.2.2
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

Successfully merging this pull request may close these issues.

2 participants