Add build test status badge to the readme

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.37k stars 358 forks source link

Add build test status badge to the readme #382

Closed Jaeyoung-Lim closed 3 years ago

Jaeyoung-Lim commented 3 years ago

Problem Description This PR adds a build status badge to the read me, which will automatically show the build status (Badge shown below)

Currently it appears as broken since the build test is broken for ROS Noetic.

Therefore disabling build test for ROS Noetic is temporarily for now

tomlankhorst commented 3 years ago

GitHub already shows this status with every commit. The badge is superfluous.

Jaeyoung-Lim commented 3 years ago

@tomlankhorst Not sure if I fully agree, the status badge shows the status of a specific action, while the github commit shows the result of the whole CI

Given that we also have jenkins running,(which is currently broken) I would say that the failure coming from jenkins is misleading

tomlankhorst commented 3 years ago

Thanks for the explanation.

Jaeyoung-Lim commented 3 years ago

@alexmillane Thanks!