Skip to content
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

Added travis CI integration #306

Closed
wants to merge 2 commits into from
Closed

Conversation

Srokap
Copy link

@Srokap Srokap commented Jul 9, 2016

  • Setup AWS S3 buckets for testing.
  • Need to encode AWS_KEY and AWS_SECRET env vars using travis encrypt for the original repo, see https://docs.travis-ci.com/user/encryption-keys/
  • Need to point all badges to the main organization
  • Need to enable builds on TravisCI for main organization and set concurrency to one job at a time (in Travis UI settings)

See https://travis-ci.org/Srokap/knox for the tests running on my fork.
See https://coveralls.io/github/Srokap/knox for how the coveralls integration looks like.

For testing I chose oldest branch I was able to run tests on (the 0.8 defined in package.json was failing on npm install), LTS and current version which is 6. For now we run them in series, so didn't want to add more than I felt was necessary. If we created separate buckets for them, they could run in parallel.

@Srokap Srokap force-pushed the travis branch 7 times, most recently from 16d12e9 to 392d68f Compare July 9, 2016 20:50
@Srokap
Copy link
Author

Srokap commented Aug 8, 2017

Tomorrow passes one year since I created this PR and it's obvious that there's no interest in it. Closing.

@Srokap Srokap closed this Aug 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant