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

Add unit-tests #106

Open
damianooldoni opened this issue Mar 8, 2021 · 0 comments
Open

Add unit-tests #106

damianooldoni opened this issue Mar 8, 2021 · 0 comments

Comments

@damianooldoni
Copy link
Member

damianooldoni commented Mar 8, 2021

Continuous integration is powerful. It would be even more powerful if we have tests for our functions. I try to do it immediately while writing the function itself or immediately after before starting a PR. In this way, we are more confident nothing breaks. I don't want to force all of us to write tests for the past as it will take a lot of time. My proposal is the following;
Do you write a new function or do you modify an existing one? Add unit-tests to your to-do-checklist before writing a PR.

There are some tests already in the ./tests/testthat/ directory you can use as examples.

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

1 participant