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

Add travis and code coverage #6

Closed
Tracked by #8
dmehra opened this issue Feb 9, 2016 · 1 comment
Closed
Tracked by #8

Add travis and code coverage #6

dmehra opened this issue Feb 9, 2016 · 1 comment
Assignees

Comments

@dmehra
Copy link

dmehra commented Feb 9, 2016

This repo is missing a .travis.yml so its tests don't run in travis - an omission?

Also, add code coverage.

See example in https://github.com/juttle/juttle-elastic-adapter
Once added, review the coverage report to see where we have holes in unit tests, and create issue to add missing tests if any.

@demmer
Copy link

demmer commented Feb 22, 2016

Assigning to @davidvgalbraith so we can get some test coverage on the adapter.

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

No branches or pull requests

3 participants