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 part of 10 to 11 #807

Merged
merged 9 commits into from
Dec 29, 2021
Merged

Merge part of 10 to 11 #807

merged 9 commits into from
Dec 29, 2021

Conversation

scpeters
Copy link
Member

➡️ Forward port

Port sdf10 as of 6b53f95 (#789) to sdf10. I'm forward-porting this separately from #780 to simplify that forward port.

Branch comparison: sdf11...6b53f95

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

@scpeters scpeters requested a review from azeey as a code owner December 29, 2021 19:06
@github-actions github-actions bot added the 🏢 edifice Ignition Edifice label Dec 29, 2021
Account for new behavior of nested models and :: syntax.

Signed-off-by: Steve Peters <[email protected]>
@codecov-commenter
Copy link

Codecov Report

Merging #807 (bff502f) into sdf11 (24a2782) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            sdf11     #807   +/-   ##
=======================================
  Coverage   88.89%   88.89%           
=======================================
  Files          73       73           
  Lines       11072    11072           
=======================================
  Hits         9843     9843           
  Misses       1229     1229           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 24a2782...bff502f. Read the comment docs.

@scpeters scpeters merged commit 5c6e870 into gazebosim:sdf11 Dec 29, 2021
@scpeters scpeters deleted the merge_9_to_11 branch December 29, 2021 19:53
@osrf-triage
Copy link

This pull request has been mentioned on Gazebo Community. There might be relevant details there:

https://community.gazebosim.org/t/new-ignition-releases-2022-03-25-fortress-edifice-citadel/1343/1

@osrf-triage
Copy link

This pull request has been mentioned on Gazebo Community. There might be relevant details there:

https://community.gazebosim.org/t/new-ignition-releases-2022-04-13-fortress-edifice/1367/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏢 edifice Ignition Edifice
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants