robotology / whole-body-estimators

YARP devices that implement estimators for humanoid robots.
26 stars 12 forks source link

Add Github Workflow to check compilation in Windows, Ubuntu and MacOS (latest) #47

Closed prashanthr05 closed 4 years ago

prashanthr05 commented 4 years ago

As a consequence of adding workflows,

Additionally,

prashanthr05 commented 4 years ago

Maybe we add a status badge in the README file.

How did you do that?

HosameldinMohamed commented 4 years ago

Maybe we add a status badge in the README file.

How did you do that?

I've copied it & modified from another working repo :smile: .

You can modify the link of the badge:

![GitHub Actions Status](https://github.com/prashanthr05/whole-body-estimators/workflows/C++%20CI%20Workflow/badge.svg) 
prashanthr05 commented 4 years ago

Then maybe we have to merge to devel and copy the badge of robotology remote rather than prashanthr05 fork?

HosameldinMohamed commented 4 years ago

Then maybe we have to merge to devel and copy the badge of robotology remote rather than prashanthr05 fork?

Yeah, we can do that.

fjandrad commented 4 years ago

Sorry for the delay, interesting work I could not have really reviewed it since I do not now these workflow things yet. But looking at the code was useful to start getting acquainted. Nice @prashanthr05 explanations and comments are quite clear.

prashanthr05 commented 4 years ago

Actually, it was only a very slightly modified version of the workflow in iDynTree and bipedal-locomotion-controllers.