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 --repo option to ipfs init. #2526

Closed
wants to merge 1 commit into from

Conversation

dignifiedquire
Copy link
Member

Closes #1296

License: MIT
Signed-off-by: dignifiedquire dignifiedquire@gmail.com

@dignifiedquire dignifiedquire force-pushed the feat/init-repo branch 2 times, most recently from 1476eb0 to e2bc6da Compare April 4, 2016 17:48
@RichardLitt
Copy link
Member

LGTM from a copy perspective.

@whyrusleeping
Copy link
Member

@dignifiedquire you broke everything. everything is broken. why is everything broken?

@dignifiedquire
Copy link
Member Author

@whyrusleeping hmm I don't know 😿 😭

@whyrusleeping whyrusleeping added the need/author-input Needs input from the original author label May 14, 2016
@Kubuxu
Copy link
Member

Kubuxu commented May 15, 2016

@dignifiedquire could you rebase it on top of current master.

@dignifiedquire dignifiedquire self-assigned this May 15, 2016
@ghost
Copy link

ghost commented Jul 21, 2016

What's the intention here? This only renames --config to --repo.

Closes ipfs#1296

License: MIT
Signed-off-by: dignifiedquire <dignifiedquire@gmail.com>
@@ -61,13 +61,15 @@ TOOL COMMANDS
Use 'ipfs <command> --help' to learn more about each command.

ipfs uses a repository in the local file system. By default, the repo is located
at ~/.ipfs. To change the repo location, set the $IPFS_PATH environment variable:
at ~/.ipfs. To change the repo location, you can either use --repo or set the $IPFS_PATH environment variable:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is too wide -- should fit 72 chars I think (cc @RichardLitt)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should! We need to make a tool to do this automatically.

@dignifiedquire dignifiedquire added need/community-input Needs input from the wider community and removed need/author-input Needs input from the original author labels Jul 27, 2016
@whyrusleeping
Copy link
Member

Closing due to inactivity. Please reopen if you plan on continuing with this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need/community-input Needs input from the wider community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants