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 14 -> main #1391

Merged
merged 5 commits into from
Apr 5, 2024
Merged

Merge 14 -> main #1391

merged 5 commits into from
Apr 5, 2024

Conversation

iche033
Copy link
Contributor

@iche033 iche033 commented Apr 4, 2024

➡️ Forward port

Port sdf14 to main

Branch comparison: main...sdf14

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

azeey and others added 4 commits March 14, 2024 17:41
Signed-off-by: Addisu Z. Taddese <[email protected]>
This fixes an error when building https://github.com/gazebo-release/sdformat_vendor/ in the ROS buildfarm where `CMAKE_STAGING_PREFIX` is used

Signed-off-by: Addisu Z. Taddese <[email protected]>
Signed-off-by: Addisu Z. Taddese <[email protected]>
@iche033 iche033 requested review from azeey and scpeters as code owners April 4, 2024 18:01
@github-actions github-actions bot added the 🏛️ ionic Gazebo Ionic label Apr 4, 2024
Copy link

codecov bot commented Apr 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.45%. Comparing base (c8e9891) to head (1860956).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1391   +/-   ##
=======================================
  Coverage   92.45%   92.45%           
=======================================
  Files         134      134           
  Lines       17820    17820           
=======================================
  Hits        16475    16475           
  Misses       1345     1345           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you make the same changes in the sdf/1.12 directory?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yep done.

Signed-off-by: Ian Chen <[email protected]>
@iche033 iche033 force-pushed the merge_14_main_20240404 branch from 4f77316 to 1860956 Compare April 4, 2024 21:29
@iche033 iche033 requested a review from azeey April 5, 2024 17:00
@iche033 iche033 merged commit 7af63cb into main Apr 5, 2024
10 checks passed
@iche033 iche033 deleted the merge_14_main_20240404 branch April 5, 2024 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏛️ ionic Gazebo Ionic
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants