ethz-asl / voxblox

A library for flexible voxel-based mapping, mainly focusing on truncated and Euclidean signed distance fields.
BSD 3-Clause "New" or "Revised" License
1.29k stars 350 forks source link

Add build tests using github actions #380

Closed Jaeyoung-Lim closed 2 years ago

Jaeyoung-Lim commented 2 years ago

Problem Description The Jenkins CI that is currently being used doesn't seem to be transparent and does not support Ubuntu Focal. Most PR's build test appear to be broken therefore does not give very useful feedback on evaluating PRs

Proposed Solution This PR adds a build test using github actions of this package for the following ROS versions

By adding the build tests some of the dependencies that were either missing or self including (voxblox was also part of the dependency of this package) has been fixed.

Testing Since github actions are not being triggered before this PR getting merged, the output can be found running on the fork: https://github.com/Jaeyoung-Lim/voxblox/pull/1

@alexmillane Is there anychance that the github actions are disabled in this repo?

Additional Context

ethzasl-jenkins commented 2 years ago

Can one of the admins verify this patch?

alexmillane commented 2 years ago

@Jaeyoung-Lim Thanks for this. I think this is a good idea. The fact the build is broken is blocking us from merging some important changes #379, so we should really fix this, and this seems like the way to go.

I just shot you an email to sort out some details.

Jaeyoung-Lim commented 2 years ago

@alexmillane Thanks! If you have admin rights, could you enable github actions so that the builds would start running?

helenol commented 2 years ago

Add to whitelist

Jaeyoung-Lim commented 2 years ago

@helenol Thanks!