Skip to content

Merge ign-msgs5 ➡️ ign-msgs8#264

Merged
chapulina merged 2 commits intoign-msgs8from
chapulina/5_to_8
Jun 23, 2022
Merged

Merge ign-msgs5 ➡️ ign-msgs8#264
chapulina merged 2 commits intoign-msgs8from
chapulina/5_to_8

Conversation

@chapulina
Copy link
Contributor

➡️ Forward port

Port ign-msgs5 ➡️ ign-msgs8

Branch comparision: ign-msgs8...ign-msgs5

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

mabelzhang and others added 2 commits June 15, 2022 01:42
Signed-off-by: Mabel Zhang <mabel@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>

Co-authored-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@codecov
Copy link

codecov bot commented Jun 23, 2022

Codecov Report

Merging #264 (70df8f4) into ign-msgs8 (d426ba7) will not change coverage.
The diff coverage is n/a.

❗ Current head 70df8f4 differs from pull request most recent head 461e598. Consider uploading reports for the commit 461e598 to get more accurate results

@@            Coverage Diff             @@
##           ign-msgs8     #264   +/-   ##
==========================================
  Coverage      85.38%   85.38%           
==========================================
  Files              9        9           
  Lines            951      951           
==========================================
  Hits             812      812           
  Misses           139      139           

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 d426ba7...461e598. Read the comment docs.

@chapulina chapulina merged commit 1ab6c70 into ign-msgs8 Jun 23, 2022
@chapulina chapulina deleted the chapulina/5_to_8 branch June 23, 2022 18:02
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.

4 participants