pablogs9 commented 3 years ago

CI is not passing because it relies on the rmw package installation of the Rolling release docker, it is not updated yet or https://github.com/ros2/rmw/pull/294 is not in bloom release.

Wait for merge until CI pass.

pablogs9 commented 3 years ago

Marie Kondo

github-actions[bot] commented 3 years ago

Static memory analysis

Default configuration

MTU: 512 B Input buffer size: 2048 B Input history: 4 Output buffer size: 2048 B Output history: 4

Entity	Qty	Size per unit
Context	1	5584 B
Topic	12	64 B
Service	4	112 B
Client	4	104 B
Subscription	4	200 B
Publisher	8	216 B
Node	4	56 B
Static input buffer	8	2120 B

TOTAL: 25564 B

codecov-commenter commented 3 years ago

Codecov Report

Merging #122 (7bb5792) into main (bf3c0e1) will decrease coverage by 0.13%. The diff coverage is 0.00%.

:exclamation: Current head 7bb5792 differs from pull request most recent head 979ce44. Consider uploading reports for the commit 979ce44 to get more accurate results

@@            Coverage Diff             @@
##             main     #122      +/-   ##
==========================================
- Coverage   50.43%   50.29%   -0.14%     
==========================================
  Files          35       36       +1     
  Lines        1507     1511       +4     
  Branches      306      306              
==========================================
  Hits          760      760              
- Misses        523      527       +4     
  Partials      224      224

Impacted Files	Coverage Δ
...microxrcedds_c/src/rmw_get_endpoint_network_flow.c	`0.00% <0.00%> (ø)`

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update bf3c0e1...979ce44. Read the comment docs.

micro-ROS / rmw_microxrcedds

Add unique network flow API #122

Static memory analysis

Codecov Report