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

fix(cli): Tell user to init repo if not initialized when starting daemon#633

Merged
daviddias merged 1 commit into
masterfrom
deamon-init-msg
Nov 29, 2016
Merged

fix(cli): Tell user to init repo if not initialized when starting daemon#633
daviddias merged 1 commit into
masterfrom
deamon-init-msg

Conversation

@victorb

@victorb victorb commented Nov 29, 2016

Copy link
Copy Markdown
Member

From #631

Had to create PR with branch from ipfs/js-ipfs instead of my fork, since running Sauce Labs would fail otherwise. See https://docs.travis-ci.com/user/pull-requests/#Pull-Requests-and-Security-Restrictions

@victorb victorb added the status/in-progress In progress label Nov 29, 2016
@victorb victorb mentioned this pull request Nov 29, 2016
@victorb

victorb commented Nov 29, 2016

Copy link
Copy Markdown
Member Author

@diasdavid tests were failing because 1) I set the timeout to one second while Travis is a lot slower to run than that, so removed my manual timeout to use the default one instead, 2) Sauce Labs can't authenticate if the branch comes from a fork, so I had to push the branch to ipfs/js-ipfs instead (opened issue in aegir about this here: ipfs/aegir#76 )

So now everything should work, please review again :)

@daviddias daviddias merged commit 237da20 into master Nov 29, 2016
@daviddias daviddias deleted the deamon-init-msg branch November 29, 2016 11:53
@daviddias daviddias removed status/in-progress In progress labels Nov 29, 2016
@daviddias

Copy link
Copy Markdown
Member

Thank you <3

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