gazebosim / gz-transport

Transport library for component communication based on publication/subscription and service calls.
https://gazebosim.org
Apache License 2.0
29 stars 36 forks source link

Merge gz-transport12 ➡️ main #414

Closed scpeters closed 1 year ago

scpeters commented 1 year ago

➡️ Forward port

Port gz-transport12 ➡️ main

Branch comparison: https://github.com/gazebosim/gz-transport/compare/main...gz-transport12

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

codecov[bot] commented 1 year ago

Codecov Report

Merging #414 (05d6849) into main (4a29c11) will not change coverage. The diff coverage is 100.00%.

:exclamation: Current head 05d6849 differs from pull request most recent head bfa1f0f. Consider uploading reports for the commit bfa1f0f to get more accurate results

@@           Coverage Diff           @@
##             main     #414   +/-   ##
=======================================
  Coverage   87.45%   87.45%           
=======================================
  Files          60       60           
  Lines        5291     5291           
=======================================
  Hits         4627     4627           
  Misses        664      664           
Impacted Files Coverage Δ
include/gz/transport/Discovery.hh 84.64% <100.00%> (ø)
mjcarroll commented 1 year ago

@osrf-jenkins retest this please

caguero commented 1 year ago

@scpeters , I just merged a minor change in gz-transport12 in a separate PR.

scpeters commented 1 year ago

@scpeters , I just merged a minor change in gz-transport12 in a separate PR.

I just force-pushed a new merge commit that should include your changes, thanks for the heads up