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

tests: Ensure removed SDFormat elements raise errors in newer versions #490

Merged
merged 2 commits into from
Feb 12, 2021

Conversation

EricCousineau-TRI
Copy link
Collaborator

@EricCousineau-TRI EricCousineau-TRI commented Feb 11, 2021

@github-actions github-actions bot added the 🏢 edifice Ignition Edifice label Feb 11, 2021
@EricCousineau-TRI
Copy link
Collaborator Author

Copy link
Collaborator

@brawner brawner left a comment

Choose a reason for hiding this comment

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

Nice, looks good to me.

Signed-off-by: Eric Cousineau <[email protected]>
Copy link
Collaborator

@jennuine jennuine left a comment

Choose a reason for hiding this comment

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

LGTM!

@brawner
Copy link
Collaborator

brawner commented Feb 12, 2021

@EricCousineau-TRI Do you have merge privileges on this repo, or do you need someone to merge for you?

@EricCousineau-TRI
Copy link
Collaborator Author

Nope, no merge privileges (and it's good that I don't have 'em :P) - can you merge it?

@scpeters scpeters merged commit 83fe424 into gazebosim:master Feb 12, 2021
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.

//joint/axis/use_parent_model_frame does not fail fast or issue a warning in SDFormat 1.7?
4 participants