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

[rolling] Update maintainers - 2022-11-07 #1017

Merged
merged 2 commits into from
Nov 15, 2022
Merged

Conversation

audrow
Copy link
Member

@audrow audrow commented Nov 7, 2022

This PR updates the maintainers and code owners. The new maintainers are:

This PR was made with https://github.com/audrow/update-ros2-repos.

@clalancette
Copy link
Contributor

xmllint is upset about the order of tags in this one. In particular, <author> tags need to come after the <license> tag.

I can fix this by hand, but is there something we need to update in the script for this?

rcl/package.xml Outdated
Comment on lines 14 to 16
<author email="ivanpauno@ekumenlabs.com">Ivan Paunovic</author>
<author email="jacob@openrobotics.org">Jacob Perron</author>
<author email="william@openrobotics.org">William Woodall</author>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<author email="ivanpauno@ekumenlabs.com">Ivan Paunovic</author>
<author email="jacob@openrobotics.org">Jacob Perron</author>
<author email="william@openrobotics.org">William Woodall</author>
<author email="jacob@openrobotics.org">Jacob Perron</author>

There's no need to add us here if we're still maintainers

Copy link
Member

Choose a reason for hiding this comment

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

same in the other files

Signed-off-by: Audrow Nash <audrow@openrobotics.org>
Signed-off-by: Audrow Nash <audrow@openrobotics.org>
@audrow audrow merged commit e4f7e13 into rolling Nov 15, 2022
@audrow audrow deleted the audrow/update-maintainers branch November 15, 2022 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants