ian-andrich / PythonPengines

A Python interface to the Prolog pengines library. Makes it exceedingly easy to call out to SWI-Prolog within Python!
MIT License
12 stars 9 forks source link

Travis CI Future Modifications #18

Open ian-andrich opened 5 years ago

ian-andrich commented 5 years ago

@cmungall Do we have the need to test this on Python 2.7? Travis CI is currently testing Python versions 3.5-3.7.

Do we want to test on windows and mac as well? Adding docker support will take an hour or two of work for mac. The noise in the docker file enables easy testing on multiple oses. Using swivm and pulling out the logic from docker might make it easier.

If we go the swivm route, would it be worthwhile testing on multiple swipl versions?

cmungall commented 5 years ago

I don't think we need to check Python 2.x. I don't have strong opinions about testing on different prolog versions. I had in mind just testing using a standard linux distro, I have no experience of testing mac and windows via travis. For me the main thing is testing the logic, rather than all possible builds but YMMV

On Sat, Aug 17, 2019 at 12:46 AM ian-andrich notifications@github.com wrote:

@cmungall https://github.com/cmungall Do we have the need to test this on Python 2.7? Travis CI is currently testing Python versions 3.5-3.7.

Do we want to test on windows and mac as well? Adding docker support will take an hour or two of work for mac. The noise in the docker file enables easy testing on multiple oses. Using swivm and pulling out the logic from docker might make it easier.

If we go the swivm route, would it be worthwhile testing on multiple swipl versions?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ian-andrich/PythonPengines/issues/18?email_source=notifications&email_token=AAAMMOLJ6QJ77RJUOC74X4DQE6UFJA5CNFSM4IMOUBJKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HFYTHDQ, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAMMOOLE6QAOAJXE46EEQLQE6UFJANCNFSM4IMOUBJA .