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: 13 -> main #454

Closed iche033 closed 12 months ago

iche033 commented 12 months ago

➡️ Forward port

Port gz-transport13 to main

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

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

codecov[bot] commented 12 months ago

Codecov Report

Merging #454 (5b12bbe) into main (685ad65) will increase coverage by 0.42%. Report is 20 commits behind head on main. The diff coverage is 83.92%.

:exclamation: Current head 5b12bbe differs from pull request most recent head b012e82. Consider uploading reports for the commit b012e82 to get more accurate results

@@            Coverage Diff             @@
##             main     #454      +/-   ##
==========================================
+ Coverage   87.33%   87.76%   +0.42%     
==========================================
  Files          60       59       -1     
  Lines        5211     5696     +485     
==========================================
+ Hits         4551     4999     +448     
- Misses        660      697      +37     
Files Coverage Δ
include/gz/transport/Discovery.hh 87.00% <100.00%> (+3.36%) :arrow_up:
include/gz/transport/HandlerStorage.hh 100.00% <100.00%> (ø)
include/gz/transport/TopicStorage.hh 100.00% <100.00%> (ø)
log/src/Log.cc 79.45% <ø> (+1.48%) :arrow_up:
src/NetUtils.cc 75.22% <ø> (+3.26%) :arrow_up:
src/NodeOptions.cc 100.00% <ø> (+3.77%) :arrow_up:
src/NodeShared.cc 78.99% <100.00%> (+1.22%) :arrow_up:
src/cmd/gz.cc 95.20% <100.00%> (+0.59%) :arrow_up:
src/Publisher.cc 99.40% <94.11%> (-0.60%) :arrow_down:
src/Node.cc 91.12% <85.71%> (+0.31%) :arrow_up:
... and 1 more

... and 26 files with indirect coverage changes

azeey commented 12 months ago

Not sure why the Jammy job didn't run. I've triggered it manually: Build Status

j-rivero commented 12 months ago

@osrf-jenkins run tests

iche033 commented 12 months ago

CI looks good now. Merging.