Feature/add continuous integration/circleci #39
Conversation
We use circle because it is very simple to setup and took less than 20 mins to implement and test. Here we use a rolling docker image for node.js, so we get constantly test against latest. At some point it may be prudent to schedule a build so we can disambiguate between code-fails and node upgrade fails.
Although this step is clearly listed in the docs, this caught me out when running through the examples because, at the time of writing, non-0.18.0/0.20.0 are the only versions that appear to work with this framework, not latest. Whilst it is clearly expressed as an npm peerDependency and using the most recent version of node-red does print errors about this, new people to the framework may be put off by this - as it *appear* that it doesn't work out-of-the-box. Part of the reason for this PR is to demonstrate how to use these tools.
|
|
|
Can I get an update on this? Or should I just close it? |
|
Hi, sorry to have overlooked this for so long. We use Travis in the main project repository. Not keen to introduce another 3rd party service as a dependency for us to have to maintain. |
|
Grand, thanks for the update. I shall close and log an issue to get a badge added to the main repo. |
|
Ah, I misunderstood your reply. I was going through my history, according to #7 you have travis in the main repo, which is not this one. So would you be amiable to having a travis setup on this repo? I want to see that the system is green, and know how to test it, before contributing. |
|
Yup, a Travis setup would be welcome. |
|
Just a quick update whilst I try to find time to look at this again. When I ran travis against this repo a few days back it failed. I cloned the main repo's travis config, but I've not yet done any due diligence (or any other work) on whether its my changes or the repo that's at fault. But you can have a look in the mean-time: p.s. the main repo seems to be failing which would be nice to see fixed https://travis-ci.org/github/node-red/node-red/jobs/718826913 |
|
Thanks for the update. We're well aware of the Travis status of the main repo and the activity going on to resolve that. |
This adds CircleCI auto-test and running of the examples to node-red-node-test-helper.
Example of it running against my repo:
https://circleci.com/gh/doublethefish/node-red-node-test-helper
In addition to the great benefits of using CI, part of the intent here is to demonstrate how to run the tests and examples, bolstering the pretty good documentation.