diff-match-patch Continuous Integration

It would help if we sets up continuous integration, to make sure that pull requests won’t break anything.

Feb 21 '18 01:02 FranklinYu

Would travis-ci work ? Or is there a preference on the ci tool ?

May 21 '18 03:05 mithuns

@mithuns Travis CI is the most popular one, but there are many out there (like Semaphore, CircleCI, Codeship). Any of them would help.

Note that Travis CI (and many others) doesn't support Windows, so typically they are combined with AppVeyor, the only Windows player in town (AFAIK).

May 21 '18 15:05 FranklinYu

Okay, but I think I would wait till https://github.com/google/diff-match-patch/pull/2 gets merged, for others, I am not so sure, like the cpp needs a csproj and the c# code would need a .sln file in order for them to be buildable, right. May be we can do separate pull requests for each of them. Suggestions ?

May 23 '18 15:05 mithuns

.csproj is for C#. The current C++ implementation seems to use QMake to build. I prefer calling it "Qt implementation" than a "C++ implementation" because Qt is a hard compile time requirement.

At least we can start testing Python versions? The JavaScript version also needs to be updated after #13.

May 23 '18 18:05 FranklinYu

@franklinyu hey, so I created something to try out, the python builds are showing success, but seems like no tests are being run, can you please take a look and suggest something. https://travis-ci.org/mithuns/diff-match-patch/jobs/384997323

May 29 '18 02:05 mithuns

@mithuns I think it has to do with test discovery. I suggest not investing too much time in this unless @NeilFraser show interest, because IIUC he is the maintainer of this repository.

May 29 '18 03:05 FranklinYu