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.

Sign up for GitHub

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

Jump to bottom

7 ➡️ 8 (main) #182

Merged
merged 8 commits into from
Sep 13, 2021
Merged

7 ➡️ 8 (main) #182

merged 8 commits into from
Sep 13, 2021

Conversation

chapulina
Copy link
Contributor

➡️ Forward port

Port ign-msgs7 to ign-msgs8

Branch comparison: ign-msgs8...ign-msgs7

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

🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸

j-rivero and others added 8 commits July 20, 2021 11:04
@j-rivero
Detect ign instead of using cmake module to check for ignition-tools (#…
e32a8bc 
…166)

Signed-off-by: Jose Luis Rivero <jrivero@osrfoundation.org>
@francocipollone
Adds PerformanceSensorMetrics proto message. (#172)
47dae0f 
* Adds PerformanceSensorMetrics proto message.

Signed-off-by: Franco Cipollone <franco.c@ekumenlabs.com>

* Use Double for fps field.

Signed-off-by: Franco Cipollone <franco.c@ekumenlabs.com>
@chapulina
5 ➡️ 6
d4af81f 
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina
5 ➡️ 6 (#174)
275afa7 
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina
🎈 6.6.0 (#176)
e559c9e 
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina
6 ➡️ 7
95ed95e 
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina
6 ➡️ 7 (#180)
e746ae7 
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina
7 ➡️ 8 (main)
689c871 
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina chapulina added the beta Targeting beta release of upcoming collection label Sep 10, 2021
@chapulina chapulina requested a review from caguero as a code owner September 10, 2021 22:59
@github-actions github-actions bot added the 🏯 fortress Ignition Fortress label Sep 10, 2021
@chapulina chapulina mentioned this pull request Sep 10, 2021
Closed
@codecov
Copy link

codecov bot commented Sep 10, 2021 •
edited
Loading

Codecov Report

Merging #182 (689c871) into main (57d13bb) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #182   +/-   ##
=======================================
  Coverage   84.78%   84.78%           
=======================================
  Files           7        7           
  Lines         828      828           
=======================================
  Hits          702      702           
  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 57d13bb...689c871. Read the comment docs.

@chapulina chapulina merged commit 1275b5d into main Sep 13, 2021
@chapulina chapulina deleted the chapulina/7_to_8 branch September 13, 2021 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@caguero caguero caguero approved these changes

Assignees
No one assigned
Labels
beta Targeting beta release of upcoming collection 🏯 fortress Ignition Fortress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@chapulina @caguero @j-rivero @francocipollone