Skip to content
This repository has been archived by the owner on Jul 11, 2019. It is now read-only.

Run examples tests directly from root package.json (instead of travis) #105

Merged
merged 2 commits into from
May 14, 2018

Conversation

theethernaut
Copy link
Contributor

Previously, the example's tests were ran by travis builds. this caused confusion with npm tests in root passing, but not passing in CI. This PR makes it more clear that example tests are ran together with the root tests.

@theethernaut theethernaut added the status:review PR waiting for review label May 14, 2018
@theethernaut theethernaut added this to the v0.1.0 (Kernel MVP) milestone May 14, 2018
@theethernaut theethernaut requested a review from maraoz May 14, 2018 15:34
@theethernaut theethernaut added this to Backlog in MVP via automation May 14, 2018
@maraoz
Copy link
Contributor

maraoz commented May 14, 2018

Thanks! Waiting for travis and I'll merge

@frangio
Copy link
Contributor

frangio commented May 14, 2018

I think this is a bit messy. That there is a script which is running npm install, because it has dependencies that are not covered by zos-lib's dependencies. This probably messes up with Travis' caching of dependencies and slows our build.

I can't think of an alternative atm so we can merge it until we have time to make it better.

@theethernaut
Copy link
Contributor Author

@frangio Would you like to create an issue to tidy this up in the future?

@maraoz maraoz merged commit b8f1764 into master May 14, 2018
MVP automation moved this from Backlog to Done May 14, 2018
@martriay martriay deleted the example-tests-npm branch May 14, 2018 17:46
spalladino pushed a commit to OpenZeppelin/openzeppelin-sdk that referenced this pull request Aug 1, 2018
…ts-npm

Run examples tests directly from root package.json (instead of travis)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:review PR waiting for review
Projects
No open projects
MVP
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants