search

roboticslab-uc3m / yarp-devices

A place for YARP devices
https://robots.uc3m.es/yarp-devices/
9 stars 7 forks source link

Fixed and updated: testCuiAbsolute app #206

Closed rsantos88 closed 5 years ago

rsantos88 commented 5 years ago

Fixed and updated: testCuiAbsolute app

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at ?% when pulling 3d78122b90e0b329fb43b6a66cdf780a4ed4d1c2 on fix-testCuiAbsolute into 304b2455538479d21e2ecfc77678e596f359353d on develop.

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at ?% when pulling 3d78122b90e0b329fb43b6a66cdf780a4ed4d1c2 on fix-testCuiAbsolute into 304b2455538479d21e2ecfc77678e596f359353d on develop.

PeterBowman commented 5 years ago

It turns out we merged a patch that does not compile on newer YARP releases (check Travis builds), let's avoid this in the future.

Edit: it's actually my fault, stuff like this shouldn't have gone into production.

https://github.com/roboticslab-uc3m/yarp-devices/pull/206/files#diff-5ead1331a2028af7ab27fc50eecf59fcR10

PeterBowman commented 5 years ago

Fixed at https://github.com/roboticslab-uc3m/yarp-devices/commit/a023145ed8a77cbeca0109c405855179654bf282, this is being documented at https://github.com/roboticslab-uc3m/developer-manual/issues/19 (FYI @rsantos88).