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

Tests need coverage assessment #83

Closed
ardunn opened this issue Oct 2, 2018 · 2 comments
Closed

Tests need coverage assessment #83

ardunn opened this issue Oct 2, 2018 · 2 comments
Assignees

Comments

@ardunn
Copy link
Contributor

ardunn commented Oct 2, 2018

We should add coverage reports to make sure (1) we have enough tests and (2) the tests we do have are useful

@utf utf self-assigned this Oct 9, 2018
@utf
Copy link
Member

utf commented Oct 9, 2018

We can't do this for free until matbench is made public. At what stage do we want that to happen?

@ardunn
Copy link
Contributor Author

ardunn commented Oct 9, 2018

Maybe once we get a few more todos done and have a paper underway? I think we should at least have the software in a better place (according to the Plan) before making public

@ardunn ardunn closed this as completed Nov 16, 2018
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

2 participants