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

TST: Make travis look at test coverage. #345

Merged
merged 8 commits into from
Apr 13, 2014

Conversation

arokem
Copy link
Contributor

@arokem arokem commented Apr 11, 2014

Adapted from nipy. The idea is to have travis produce the coverage report every time it runs the tests. Let's see if it works :-)

@Garyfallidis
Copy link
Contributor

--with-coverage command not found. See Travis details.

@arokem
Copy link
Contributor Author

arokem commented Apr 11, 2014

OK - I will tweak.

On Fri, Apr 11, 2014 at 11:28 AM, Eleftherios Garyfallidis <
notifications@github.com> wrote:

--with-coverage command not found. See Travis details.


Reply to this email directly or view it on GitHubhttps://github.com//pull/345#issuecomment-40238402
.

@arokem
Copy link
Contributor Author

arokem commented Apr 13, 2014

After much tweaking:

https://travis-ci.org/nipy/dipy/jobs/22866860

No more excuses! :-)

cc @Garyfallidis

On Fri, Apr 11, 2014 at 12:06 PM, Ariel Rokem arokem@gmail.com wrote:

OK - I will tweak.

On Fri, Apr 11, 2014 at 11:28 AM, Eleftherios Garyfallidis <
notifications@github.com> wrote:

--with-coverage command not found. See Travis details.


Reply to this email directly or view it on GitHubhttps://github.com//pull/345#issuecomment-40238402
.

@Garyfallidis
Copy link
Contributor

Wow, it seems we are doing really great the last years. It's mostly very old code which is not tested much and some modules which are not dipy related. GGT!!

Super!!! To 100% percent coverage and beyond!!

Garyfallidis added a commit that referenced this pull request Apr 13, 2014
TST: Make travis look at test coverage.
@Garyfallidis Garyfallidis merged commit 93bcafe into dipy:master Apr 13, 2014
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.

2 participants