search

atTC2 / leslie

UoB CS Intelligent Robotics Exercise 2
0 stars 1 forks source link

Add Travis testing #109

Closed rrama closed 7 years ago

rrama commented 7 years ago

Add Pylint testing Add Travis badge Modify setup_scripts to work with Travis

Tests should only take 6 (provided dlib doesn't update). So when you're running builds have a listen to Eminem - Rap God, twiddle your thumbs a bit and it should be done. Concurrent builds are enabled, so your commit check and PR check (checks that if merged with master nothing will break) can be run at the same time. I highly doubt our code will reach a state where only one person can run it at once (except the few clashes with Twitter stuff). Travis does use Git LFS and our scripts to install LFS fail, but that isn't a problem since it doesn't cause a bad exit, so we can just ignore that fail line in the log.

rrama commented 7 years ago

Just realised I have the badge set to .org not .com... I'll change it in a sec.

Yeah Travis didn't like doing a full dist-upgrade (shouldn't be needed though, since the install below will upgrade the packages properly) and doesn't have ~/Documents by default.

atTC2 commented 7 years ago

Fair enough, when that's done I'll approve.

rrama commented 7 years ago

@TomBClarke Done.