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

Organise scripts #91

Merged
merged 4 commits into from Sep 21, 2017

Conversation

Projects
None yet
3 participants
@mor1
Contributor

mor1 commented Sep 20, 2017

No description provided.

@haddadi

This comment has been minimized.

Show comment
Hide comment
@haddadi

haddadi Sep 20, 2017

Member

seems like we get a build fail?

container-manager@0.1.0 test /home/travis/build/me-box/databox
./node_modules/mocha/bin/mocha
sh: 1: ./node_modules/mocha/bin/mocha: not found
npm ERR! Test failed. See above for more details.
The command "npm test" exited with 1.
Done. Your build exited with 1.

Member

haddadi commented Sep 20, 2017

seems like we get a build fail?

container-manager@0.1.0 test /home/travis/build/me-box/databox
./node_modules/mocha/bin/mocha
sh: 1: ./node_modules/mocha/bin/mocha: not found
npm ERR! Test failed. See above for more details.
The command "npm test" exited with 1.
Done. Your build exited with 1.

@mor1

This comment has been minimized.

Show comment
Hide comment
@mor1

mor1 Sep 21, 2017

Contributor
Contributor

mor1 commented Sep 21, 2017

@Toshbrown

This comment has been minimized.

Show comment
Hide comment
@Toshbrown

Toshbrown Sep 21, 2017

Contributor

Yep, I'm working on testing so enabled Travis. This is not a problem merging.

Contributor

Toshbrown commented Sep 21, 2017

Yep, I'm working on testing so enabled Travis. This is not a problem merging.

@Toshbrown Toshbrown merged commit 8ceb955 into me-box:master Sep 21, 2017

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment