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

Merge gz-msgs10 ➡️ main #395

Merged
merged 17 commits into from
Oct 9, 2023
Merged

Merge gz-msgs10 ➡️ main #395

merged 17 commits into from
Oct 9, 2023

Conversation

azeey
Copy link
Contributor

@azeey azeey commented Oct 9, 2023

➡️ Forward port

Port gz-msgs10 to main
Branch comparison: main...gz-msgs10

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

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

azeey and others added 17 commits August 25, 2023 12:07
@azeey
Prepare for 9.5.0 (gazebosim#373)
ca9d66c 
Signed-off-by: Addisu Z. Taddese <[email protected]>
@mjcarroll
Fix generator expression (gazebosim#374)
0a47973 
Signed-off-by: Michael Carroll <[email protected]>
@azeey
Prepare for 10.0.0~pre1 (gazebosim#375)
7955f90 
Signed-off-by: Addisu Z. Taddese <[email protected]>
@azeey
Add pre1 suffix (gazebosim#376)
1cb2374 
Signed-off-by: Addisu Z. Taddese <[email protected]>
@mjcarroll
Default to identity quaternion when not set (gazebosim#377)
4630db1 
Signed-off-by: Michael Carroll <[email protected]>
@azeey
Prepare for 10.0.0~pre2 (gazebosim#378)
ad3ad00 
Signed-off-by: Addisu Z. Taddese <[email protected]>
@mjcarroll
Support setting (or unsetting) dllexport macros (gazebosim#379)
3aa9224 
Signed-off-by: Michael Carroll <[email protected]>
@azeey
Prepare for 10.0.0~pre3 (gazebosim#380)
b6e42df 
Signed-off-by: Addisu Z. Taddese <[email protected]>
@ahcorde
Update README.md (gazebosim#381)
50ee1e4 
Signed-off-by: Alejandro Hernández Cordero <[email protected]>
@mjcarroll
Add missing <functional> header (gazebosim#382)
74dc3b4 
Signed-off-by: Michael Carroll <[email protected]>
@mjcarroll
Return nullptr when message fails to populate (gazebosim#384)
9b6a7d8 
Signed-off-by: Michael Carroll <[email protected]>
@azeey
Merge gz-msgs9 into gz-msgs10
f655719 
Signed-off-by: Addisu Z. Taddese <[email protected]>
@azeey
Merge pull request gazebosim#387 from azeey/9_to_10
a6fbd92 
Merge gz-msgs9 ➡️  gz-msgs10
@mjcarroll @azeey
Improve message generation tutorial and example (gazebosim#386)
a731933 
Signed-off-by: Michael Carroll <[email protected]>
Signed-off-by: Addisu Z. Taddese <[email protected]>
Co-authored-by: Addisu Z. Taddese <[email protected]>
@azeey
Update changelog and prepare for pre4 (gazebosim#388)
146e399 
Signed-off-by: Addisu Z. Taddese <[email protected]>
@azeey
Prepare for 10.0.0 Release (gazebosim#389)
0472ba0 
Signed-off-by: Addisu Z. Taddese <[email protected]>
@traversaro
Do not redefine GZ_PYTHON_INSTALL_PATH if it is already defined (gaze…
0f69062 
…bosim#393)

Signed-off-by: Silvio Traversaro <[email protected]>
@azeey azeey requested a review from caguero as a code owner October 9, 2023 22:43
@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

Merging #395 (fd7f326) into main (7fc5805) will increase coverage by 1.69%.
Report is 31 commits behind head on main.
The diff coverage is 98.17%.

❗ Current head fd7f326 differs from pull request most recent head 0f69062. Consider uploading reports for the commit 0f69062 to get more accurate results

@@            Coverage Diff             @@
##             main     #395      +/-   ##
==========================================
+ Coverage   95.57%   97.26%   +1.69%     
==========================================
  Files          10       27      +17     
  Lines        1062     1170     +108     
==========================================
+ Hits         1015     1138     +123     
+ Misses         47       32      -15
Files Coverage Δ
core/generator/Generator.cc 92.64% <100.00%> (ø)
core/generator/generator_main.cc 100.00% <ø> (ø)
core/include/gz/msgs/Factory.hh 100.00% <100.00%> (ø)
core/include/gz/msgs/MessageFactory.hh 100.00% <100.00%> (ø)
core/include/gz/msgs/PointCloudPackedUtils.hh 94.91% <100.00%> (ø)
core/include/gz/msgs/convert/AxisAlignedBox.hh 100.00% <100.00%> (ø)
core/include/gz/msgs/convert/Color.hh 100.00% <100.00%> (ø)
core/include/gz/msgs/convert/DiscoveryType.hh 100.00% <100.00%> (ø)
core/include/gz/msgs/convert/FuelMetadata.hh 100.00% <100.00%> (ø)
core/include/gz/msgs/convert/GeometryType.hh 100.00% <100.00%> (ø)
... and 17 more

... and 1 file with indirect coverage changes

@iche033 iche033 merged commit f5eabb1 into gazebosim:main Oct 9, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iche033 iche033 iche033 approved these changes

@caguero caguero Awaiting requested review from caguero caguero is a code owner

Assignees
No one assigned
Labels
None yet
Projects
Core development
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@azeey @iche033 @mjcarroll @ahcorde @traversaro