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.

Sign up for GitHub

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

Jump to bottom

Coverage is not reported for the analyses #804

Closed
jbarnoud opened this issue Mar 27, 2016 · 3 comments
Closed

Coverage is not reported for the analyses #804

jbarnoud opened this issue Mar 27, 2016 · 3 comments
Labels
Continuous Integration

Comments

@jbarnoud
Copy link
Contributor

Since #743 got accepted, we mandate tests for the analysis modules. Yet, coverall does nor report the test coverage for these modules. This makes the review of PR like #803 more difficult.
@richardjgowers
Copy link
Member

Yep just needs switching back now

@jbarnoud
Copy link
Contributor Author

@richardjgowers what does it take? Is it something to change in .travis.yml? Ot in coverall config? I did not find the culprit in .travis.yml.

@jbarnoud
Copy link
Contributor Author

Nevermind, found it in .coveragerc.

jbarnoud added a commit to jbarnoud/mdanalysis that referenced this issue Mar 27, 2016
@jbarnoud
Activate coverage report for analyses
7306f17 
See issue MDAnalysis#804.
@jbarnoud jbarnoud mentioned this issue Mar 27, 2016
Merged
4 tasks
jbarnoud added a commit to jbarnoud/mdanalysis that referenced this issue Mar 27, 2016
@jbarnoud
Update the README to reflect MDAnalysis#804
f067101
orbeckst added a commit that referenced this issue Mar 28, 2016
@orbeckst
Merge pull request #806 from jbarnoud/issue-804-coverage
7a329b9 
Update the README to reflect #804
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Continuous Integration
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@orbeckst @jbarnoud @richardjgowers