-
Notifications
You must be signed in to change notification settings - Fork 100
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
Cherry-pick from sdf9 to sdf10 #372
Conversation
…im#245) Signed-off-by: Silvio Traversaro <[email protected]>
Signed-off-by: Steve Peters <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Louise Poubel <[email protected]> Co-authored-by: Steve Peters <[email protected]> Signed-off-by: Steve Peters <[email protected]>
@chapulina why didn't this get any labels? |
is it because I created the PR from my fork? |
added labels manually |
Codecov Report
@@ Coverage Diff @@
## sdf10 #372 +/- ##
==========================================
- Coverage 87.36% 87.32% -0.05%
==========================================
Files 60 60
Lines 9214 9214
==========================================
- Hits 8050 8046 -4
- Misses 1164 1168 +4
Continue to review full report at Codecov.
|
Yup, GitHub actions are really annoying with PRs from forks. But just last month they added a new feature that should fix. Here's an issue for tracking this: gazebo-tooling/pr-collection-labeler#4 |
Cherry-pick #245, #368, #369, #361 to
sdf10
. Use rebase and merge.I've created a branch that merged sdf9 -> sdf10 with resolved conflicts. Compare the diff to confirm that these changes are all that's missing from
sdf10
: