You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Feb 28, 2021. It is now read-only.
Repository navigation
This repository was archived by the owner on Feb 28, 2021. It is now read-only.
Hello, I think it will be very beneficial for this library to have unit tests which will automatically run on build and possibly use CI. What is the approach on this?
@spiray - Sorry for the delay! Last week was a bit of a mess.
So I don't have a ton of perspective on this. I do know that Jest os probably the more popular choice. Do you have a specific preference to one over the other?
Same as the above, really. Let's maybe stick to what os being used in the other p5 projects? Its circle CI right?
Yes! I am actually working on it right now. I am just setting up the environment based off of the way it is setup in the p5.js Sound library. Ill keep you updated.
Activity
spiray commentedon Jun 8, 2018
@MathuraMG @lm-n I wanted to get started with testing/CI as i think it will help a lot with making the contribution process smoother.
Here are my questions -
Please let me know what you think. I will create a separate issue for CI.
Thanks!
MathuraMG commentedon Jun 16, 2018
@spiray - Sorry for the delay! Last week was a bit of a mess.
@lm-n thoughts?
MathuraMG commentedon Jul 5, 2018
hey @spiray - just wanted to follow up on this. Do the above comments make sense to start with the tests?
spiray commentedon Jul 5, 2018
Yes! I am actually working on it right now. I am just setting up the environment based off of the way it is setup in the p5.js Sound library. Ill keep you updated.
lm-n commentedon Jul 5, 2018
Cool. I'll look at the PR soon. Just FYI https://github.com/processing/p5.js-web-editor uses Travis CI