-
Notifications
You must be signed in to change notification settings - Fork 8
Implement testing across the package #42
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
Comments
Hello! Happy to help write unit tests! |
@qpmnguyen @rafaelcatoia Did you guys want to start taking a look at this? You can see the way that testing is implemented in the covidcast R package (for example) and also read up on testing in R in general if you haven't had exposure to this already. In side conversation with @capnrefsmmat, he said he is busy for a little bit with other responsibilities, but can start taking a look perhaps mid March, so in any case I think it makes sense for Quant & Rafael to just start learning/putting things together in the meantime, if you're still up for it. Thanks! p.s. in the |
@dshemetov @qpmnguyen @rafaelcatoia Is anyone actively working on this? I'm going to re-assign if no. |
I'm not. |
I am not currently working on this. Feel free to re-assign! |
I just realized that I misread what tests need to be done; the following do not have tests: |
Now that the package is getting farther along, I think it is about time we implement testing across the package. Especially because I believe (and hope) that more others will be contributing to it in the near future.
@capnrefsmmat You are our resident stickler-for-the-rules-kind-of-guy, did you want to take the lead on this? You know I'm kidding 😛, and I would be grateful if you took this on. I think @qpmnguyen @rafaelcatoia could be willing to help as well, so if you have some ideas/scaffolding then perhaps they would be willing to implement it with you.
The text was updated successfully, but these errors were encountered: