Thanks to visit codestin.com
Credit goes to github.com

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

Conversation

@yotamberk
Copy link
Member

No description provided.

Copy link
Contributor

@Tooa Tooa left a comment

Choose a reason for hiding this comment

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

Do we need to add sudo: false in order to run tests in containers on Travis? Not sure, about the default value. Here it says "repositories enabled in 2015 or later". Is this talking about Travis enabling after 2015 or is it about the repository creation date?

What about using the cache for dependencies?

We should also consider adding the [![Build Status] badge to our readme.

Copy link
Member

@mojoaxel mojoaxel left a comment

Choose a reason for hiding this comment

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

@yotamberk Please add the npm run test command also.

@mojoaxel
Copy link
Member

mojoaxel commented Jan 6, 2017

Here is the markdown syntax to add to our readme:

[![Build Status](https://travis-ci.org/almende/vis.svg?branch=develop)](https://travis-ci.org/almende/vis)

@mojoaxel
Copy link
Member

mojoaxel commented Jan 6, 2017

I just noticed that I have removed the test folder from the project. We should re-add it now that we are actually using it :-)

@mojoaxel mojoaxel merged commit 5cb13de into visjs:develop Jan 7, 2017
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.

3 participants