-
Notifications
You must be signed in to change notification settings - Fork 66
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
Testing #8
Comments
Hey, I'm interested in working on this, and potentially other enhancements down the line. Any thoughts on what you want to use? |
Awesome. To be honest, I have no idea. I was reading through a blog about open sourcing a package when I first started building and one of the features they talked about was automatic testing. If you want to take ownership of that aspect and choose a direction, that would be great. You'll just have to teach me a bit about it along the way. |
Are you talking about doing Travis-CI integration? |
That's what I had started but haven't taken time to learn how to do it correctly. Package works but just get failure messages from Travis. Any help here would be thoroughly appreciated. |
Yes. All that good stuff. |
With the new refactor, I'm taking advantage of classing to minimize duplicate code. Once it's done I think it should be easier to write tests for each modules. Would be great to get some help on this once refactor is done. Would actually like to consider a jenkins up to periodically test package and all the calls to identify if the NBA makes any other breaking changes to their stats website. |
@bpblanken Think I finally understand class inheritance in a way that makes building the modules with much less copy and paste. Hopefully it's a bit more straight forward to set up some simple testing. |
Need to develop and integrate testing framework for the package.
The text was updated successfully, but these errors were encountered: