Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

Deamon init msg #631

Closed
wants to merge 1 commit into from
Closed

Deamon init msg #631

wants to merge 1 commit into from

Conversation

victorb
Copy link
Member

@victorb victorb commented Nov 28, 2016

Just output a similar message as go-ipfs if you run ipfs daemon before having init a repo.

@victorb victorb added the status/in-progress In progress label Nov 28, 2016
Copy link
Member

@daviddias daviddias left a comment

Choose a reason for hiding this comment

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

Tests fail:

  1) cli --all daemon gives error if user hasn't run init before:
     Error: Timeout of 1000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

Also, please follow: https://github.com/ipfs/community/blob/master/js-commit-guidelines.md

@victorb
Copy link
Member Author

victorb commented Nov 28, 2016

@diasdavid thanks for the CR! That's weird, only fails in Travis. Passes locally for me as well. I'll look it, and I'll fix the commit messages, sorry about that...

@victorb
Copy link
Member Author

victorb commented Nov 29, 2016

Closing this in favor of #633

@victorb victorb closed this Nov 29, 2016
@victorb victorb removed the status/in-progress In progress label Nov 29, 2016
@victorb victorb deleted the deamon-init-msg branch October 6, 2017 09:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants