search

gazebosim / gz-transport

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

Merge ign-transport8 ➡️ ign-transport11 #409

Closed scpeters closed 1 year ago

scpeters commented 1 year ago

➡️ Forward port

Port ign-transport8 ➡️ ign-transport11

Branch comparison: https://github.com/ignitionrobotics/ign-transport/compare/ign-transport11...ign-transport8

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

codecov[bot] commented 1 year ago

Codecov Report

Merging #409 (2778ba4) into ign-transport11 (efb9566) will decrease coverage by 0.09%. The diff coverage is 71.42%.

:exclamation: Current head 2778ba4 differs from pull request most recent head 19263e4. Consider uploading reports for the commit 19263e4 to get more accurate results

@@                 Coverage Diff                 @@
##           ign-transport11     #409      +/-   ##
===================================================
- Coverage            88.13%   88.04%   -0.09%     
===================================================
  Files                   59       59              
  Lines                 5115     5121       +6     
===================================================
+ Hits                  4508     4509       +1     
- Misses                 607      612       +5
Impacted Files Coverage Δ
include/gz/transport/RepHandler.hh 85.36% <ø> (ø)
include/gz/transport/SubscriptionHandler.hh 84.90% <ø> (ø)
src/cmd/gz.cc 93.89% <60.00%> (-1.42%) :arrow_down:
include/gz/transport/Discovery.hh 84.69% <100.00%> (-0.53%) :arrow_down:
scpeters commented 1 year ago

cherry-picking a warning fix from #408 in https://github.com/gazebosim/gz-transport/pull/409/commits/19263e4817a958daf0b43e69ebb520f6952f17cf