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

Add package.xml #694

Conversation

MaximilienNaveau
Copy link
Contributor

Add a package.xml file so one can use this package as a dependency directly.
I added a list of maintainer in this file.
It's hard to decide in practice as this project has many maintainers and contributors.
So I picked 3 names among the most commiters according to github...
If it's not politically correct tell me I could:

  • remove the list altogether and let the people contact the maintainers from the github utils.
  • add people in this list
  • remove people in this list
  • others?

Copy link
Collaborator

@nim65s nim65s left a comment

Choose a reason for hiding this comment

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

This maintainer list is mostly used by ROS buildfarm, so as long as we don't publish this inside https://github.com/ros/rosdistro/ this list shouldn't hurt anyone.

Still, yes, it could be good to define a proper list of authors / maintainers in this package in general.

package.xml Outdated Show resolved Hide resolved
package.xml Outdated Show resolved Hide resolved
@MaximilienNaveau MaximilienNaveau requested a review from nim65s May 16, 2024 14:45
@MaximilienNaveau MaximilienNaveau force-pushed the topic/mnaveau/add-package-xml branch from 817131f to 08d1f0e Compare May 22, 2024 08:25
@MaximilienNaveau
Copy link
Contributor Author

ready to be merged.

@MaximilienNaveau MaximilienNaveau self-assigned this May 22, 2024
@MaximilienNaveau MaximilienNaveau merged commit dd3e412 into jrl-umi3218:master May 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants