Skip to content

Merge 10 -> main #463

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
Merged
merged 7 commits into from
May 2, 2025
Merged

Merge 10 -> main #463

merged 7 commits into from
May 2, 2025

Conversation

scpeters
Copy link
Member

@scpeters scpeters commented May 1, 2025

➡️ Forward port

Port gz-fuel-tools10 to main

Branch comparison: main...gz-fuel-tools10

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

caguero and others added 7 commits February 12, 2025 20:11
@caguero
Prepare for 10.0.1 release (#453)
8484260 
Signed-off-by: Carlos Agüero <caguero@openrobotics.org>
@j-rivero
Update Windows badges in README.md [skip ci]
7627ec4 
Part of gazebo-tooling/release-tools#1288

Signed-off-by: Jose Luis Rivero <jrivero@honurobotics.com>
@mergify @khaledgabr77
Merge pull request #456 from khaledgabr77/fix-documentation-spelling (#…
e7ac6d4 
…457)

Fix documentation spelling

(cherry picked from commit efe9651)

Co-authored-by: Khaled Gabr <30967695+khaledgabr77@users.noreply.github.com>
@mergify @khaledgabr77
Fix code documentation spelling (#458) (#460)
d5f0123 
Signed-off-by: khaledgabr77 <khaledgabr77@gmail.com>
(cherry picked from commit c874efb)

Co-authored-by: Khaled Gabr <30967695+khaledgabr77@users.noreply.github.com>
@Rahul-K-A @ahcorde
Fix return value when download fails (#459)
79347d5 
Signed-off-by: Rahul K.A <karahul209@gmail.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
@scpeters
ci: run cppcheck, cpplint, doxygen on noble (#462)
809b185 
Fixes doxygen complaints

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
@scpeters
Merge branch 'gz-fuel-tools10' into scpeters/merge_10_main
04c8659
@scpeters scpeters requested a review from nkoenig as a code owner May 1, 2025 20:47
@scpeters scpeters changed the title Scpeters/merge 10 main Merge 10 -> main May 1, 2025
@scpeters scpeters mentioned this pull request May 1, 2025
Closed
83 tasks
@codecov Codecov
Copy link

codecov bot commented May 1, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 75.82%. Comparing base (1896341) to head (04c8659).
Report is 102 commits behind head on main.

Files with missing lines Patch % Lines
src/gz.cc 60.00% 2 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #463      +/-   ##
==========================================
+ Coverage   75.32%   75.82%   +0.50%     
==========================================
  Files          20       19       -1     
  Lines        2910     3144     +234     
==========================================
+ Hits         2192     2384     +192     
- Misses        718      760      +42

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@scpeters scpeters requested a review from iche033 May 1, 2025 22:16
@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development May 1, 2025
@scpeters scpeters merged commit 0d41801 into main May 2, 2025
11 of 12 checks passed
@scpeters scpeters deleted the scpeters/merge_10_main branch May 2, 2025 02:08
@github-project-automation github-project-automation bot moved this from In review to Done in Core development May 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ahcorde ahcorde ahcorde approved these changes

@iche033 iche033 iche033 approved these changes

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

Assignees

No one assigned

Labels

None yet

Projects

Core development
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@scpeters @ahcorde @iche033 @caguero @j-rivero @Rahul-K-A