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

Backport #1911 and #2254 to ign-gazebo6 #2453

Merged
merged 3 commits into from
Jul 11, 2024

Conversation

mjcarroll
Copy link
Contributor

Backport #1911 and #2254 to address #2384

Signed-off-by: Michael Carroll <[email protected]>
Signed-off-by: Michael Carroll <[email protected]>
@mjcarroll mjcarroll requested a review from azeey June 24, 2024 21:13
@mjcarroll mjcarroll self-assigned this Jun 24, 2024
@github-actions github-actions bot added the 🏯 fortress Ignition Fortress label Jun 24, 2024
@mjcarroll mjcarroll changed the title Backport #1911 and #2254 to ign-gazebo Backport #1911 and #2254 to ign-gazebo6 Jun 24, 2024
@azeey
Copy link
Contributor

azeey commented Jun 25, 2024

Looks like windows still has warnings. I don't the changes to ModelCommandAPI.hh from #1911 in here. That might be one issue.

@azeey azeey force-pushed the mjcarroll/backport_windows_fixes branch from 51482ec to 727bbf0 Compare July 10, 2024 20:04
@azeey
Copy link
Contributor

azeey commented Jul 10, 2024

I added some additional fixes in 727bbf0. The gz-physics related warning should be fixed by gazebosim/gz-physics#666

@azeey azeey merged commit a2a9c53 into ign-gazebo6 Jul 11, 2024
8 of 10 checks passed
@azeey azeey deleted the mjcarroll/backport_windows_fixes branch July 11, 2024 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏯 fortress Ignition Fortress
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants