Closed Jaeyoung-Lim closed 2 years ago
@Schmluk Thanks!
@Schmluk Voxblox and protobuf_catkin
now supports ROS Noetic and therefore re-enabled Noetic build for this repo
Nice! LGTM, feel free to merge!
@Schmluk I don't have merge rights! :sweat_smile:
Interesting I thought you should be allowed to once the PR is approved. I merged the PR and added you to the repo as a maintainer for the future :)
@Schmluk Thanks!
Problem Description This PR adds a build test CI for this repository using github actions.
This adds a nice sanity checks if the chain of catkin dependencies are being broken and if we need to start fixing dependency versions.
While adding the PR the following was added
package.xml
and add them as optional. I have separated the commits to show that the dependencies are broken for the simulationgrpc
is a dependency ofprotobuf_catkin
but was not part of the dependency list and was not getting pulledc++14
and higher and is being addressed in https://github.com/ethz-asl/voxblox/pull/379Since the build is not automatically triggered in the
ethz-asl
organization from forks, you can see the build results in https://github.com/Jaeyoung-Lim/mav_active_3d_planning/pull/1Additional Context