Skip to content

Added statistics message#110

Merged
nkoenig merged 4 commits intoign-msgs5from
nkoenig/statistics
Dec 19, 2020
Merged

Added statistics message#110
nkoenig merged 4 commits intoign-msgs5from
nkoenig/statistics

Conversation

@nkoenig
Copy link
Contributor

@nkoenig nkoenig commented Dec 10, 2020

Statistics message need by gazebosim/gz-transport#205
Signed-off-by: Nate Koenig nate@openrobotics.org

Signed-off-by: Nate Koenig <nate@openrobotics.org>
@nkoenig nkoenig requested a review from caguero as a code owner December 10, 2020 21:42
@github-actions github-actions bot added the 🏰 citadel Ignition Citadel label Dec 10, 2020
@codecov
Copy link

codecov bot commented Dec 10, 2020

Codecov Report

Merging #110 (258d95d) into ign-msgs5 (83f21b9) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           ign-msgs5     #110   +/-   ##
==========================================
  Coverage      84.38%   84.38%           
==========================================
  Files              7        7           
  Lines            807      807           
==========================================
  Hits             681      681           
  Misses           126      126           

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 83f21b9...258d95d. Read the comment docs.

Nate Koenig added 2 commits December 16, 2020 16:01
Signed-off-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: Nate Koenig <nate@openrobotics.org>
Copy link
Member

@scpeters scpeters left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't tested, but it looks ok to me

@nkoenig nkoenig merged commit a9f3f46 into ign-msgs5 Dec 19, 2020
@nkoenig nkoenig deleted the nkoenig/statistics branch December 19, 2020 01:48
@chapulina chapulina mentioned this pull request Dec 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏰 citadel Ignition Citadel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants