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

Cleaned up ipfs id mans #2285

Merged
merged 2 commits into from
Feb 4, 2016
Merged

Cleaned up ipfs id mans #2285

merged 2 commits into from
Feb 4, 2016

Conversation

RichardLitt
Copy link
Member

License: MIT
Signed-off-by: Richard Littauer [email protected]

@@ -19,7 +19,7 @@ import (
u "github.com/ipfs/go-ipfs/util"
)

const offlineIdErrorMessage = `ID command fails when run without daemon, we are working to fix this.
const offlineIdErrorMessage = `Id command fails when run without the daemon; we are working to fix this.
Copy link
Member

Choose a reason for hiding this comment

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

huh, this is no longer true. You can run ipfs id without a running daemon just fine

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah. You don't even use the daemon anymore; it doesn't hit it at all.

Copy link
Member

Choose a reason for hiding this comment

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

mind removing this in this PR?

Copy link
Member Author

Choose a reason for hiding this comment

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

Done.

@whyrusleeping
Copy link
Member

okay, after seeing the test failures and looking into it a little more, i see why we had that message. Its for if we try to id another peer on the network while offline

@whyrusleeping
Copy link
Member

@RichardLitt rewrote that error message in a way that makes more sense to me. Wanna look it over?

In the meantime, please run the daemon if you want to use 'ipfs id':
const offlineIdErrorMessage = `'ipfs id' currently cannot query information on remote
peers without a running daemon; we are working to fix this.
In the meantime, please run the daemon if you want to use 'ipfs id' in that way:
Copy link
Member Author

Choose a reason for hiding this comment

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

"In the meantime, if you want to query remote peers using 'ipfs id', please run the daemon:"

RichardLitt and others added 2 commits February 3, 2016 11:40
License: MIT
Signed-off-by: Richard Littauer <[email protected]>
@RichardLitt
Copy link
Member Author

LGTM. Might clash with #2300.

@RichardLitt RichardLitt added the RFM label Feb 4, 2016
whyrusleeping added a commit that referenced this pull request Feb 4, 2016
@whyrusleeping whyrusleeping merged commit 9300af5 into master Feb 4, 2016
@whyrusleeping whyrusleeping deleted the docs/cleanup-id-mans branch February 4, 2016 23:12
@whyrusleeping
Copy link
Member

@RichardLitt wanna rebase #2300?

@RichardLitt
Copy link
Member Author

@whyrusleeping Done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants