search

gazebosim / gz-msgs

Messages for Gazebo robot simulation.
https://gazebosim.org
Apache License 2.0
23 stars 47 forks source link

Merge gz-msgs9 ➡️ main #351

Closed scpeters closed 1 year ago

scpeters commented 1 year ago

➡️ Forward port

Port gz-msgs9 ➡️ main

Branch comparison: https://github.com/ignitionrobotics/ign-msgs/compare/main...gz-msgs9

Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

codecov[bot] commented 1 year ago

Codecov Report

Merging #351 (5935dcb) into main (68fc785) will not change coverage. The diff coverage is 100.00%.

:exclamation: Current head 5935dcb differs from pull request most recent head ccfbba9. Consider uploading reports for the commit ccfbba9 to get more accurate results

@@           Coverage Diff           @@
##             main     #351   +/-   ##
=======================================
  Coverage   95.43%   95.43%           
=======================================
  Files          10       10           
  Lines        1030     1030           
=======================================
  Hits          983      983           
  Misses         47       47
Impacted Files Coverage Δ
src/Generator.cc 92.75% <100.00%> (ø)
ahcorde commented 1 year ago

@osrf-jenkins run tests please!

mjcarroll commented 1 year ago

@scpeters homebrew build can't find protobuf.

scpeters commented 1 year ago

@scpeters homebrew build can't find protobuf.

it was an issue with the formula that I believe is fixed by https://github.com/osrf/homebrew-simulation/pull/2308

restarting homebrew build...

scpeters commented 1 year ago

trying to fix a jammy compiler warning in https://github.com/gazebosim/gz-msgs/pull/351/commits/ccfbba945a45a11de6c55baf8f08a7f8654fcd1e