Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge ign-transport8 ➡️ ign-transport11 #420

Merged
merged 3 commits into from
Jul 14, 2023
Merged

Conversation

caguero
Copy link
Collaborator

@caguero caguero commented Jul 13, 2023

➡️ Forward port

Port ign-transport8 ➡️ ign-transport11

Branch comparison: ign-transport11...ign-transport8

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

traversaro and others added 3 commits June 17, 2023 03:56
Signed-off-by: Silvio Traversaro <silvio@traversaro.it>
* Consider discovery initialized after two heartbeat cycles

Signed-off-by: Carlos Agüero <caguero@openrobotics.org>
@github-actions github-actions bot added the 🏯 fortress Ignition Fortress label Jul 13, 2023
@codecov
Copy link

codecov bot commented Jul 13, 2023

Codecov Report

Merging #420 (cba87d1) into ign-transport11 (65994b2) will not change coverage.
The diff coverage is 100.00%.

❗ Current head cba87d1 differs from pull request most recent head 87aa000. Consider uploading reports for the commit 87aa000 to get more accurate results

@@               Coverage Diff                @@
##           ign-transport11     #420   +/-   ##
================================================
  Coverage            88.04%   88.04%           
================================================
  Files                   59       59           
  Lines                 5121     5121           
================================================
  Hits                  4509     4509           
  Misses                 612      612           
Impacted Files Coverage Δ
include/gz/transport/Discovery.hh 84.69% <100.00%> (ø)

@caguero caguero merged commit 0e76a20 into ign-transport11 Jul 14, 2023
11 checks passed
@caguero caguero deleted the caguero_8_11 branch July 14, 2023 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏯 fortress Ignition Fortress
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants