-
Notifications
You must be signed in to change notification settings - Fork 25
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
ROS 2 port #16
Comments
That sounds great Dirk. I'll make the branch now. I can give external collaborator access as well for this repo if you give a list of developers to add. |
Great - thanks.
Let's start with me and we will see if we need more in the future. I will likely ask for the same on the |
Done. |
We (OSRF) would like to contribute a ROS 2 port of this package. Commonly we prefer to create a
ros2
branch in the upstream repository in order to keep the code as close as possible to the ROS 1 code and ease porting of patches between the branches.I just wanted to check if the maintainers of this repo would be open to this? This could either be done by creating a
ros2
branch for us and we create PRs or (if you would be comfortable with giving us write access) we could move forward on this without requiring your attention for future PRs (we would still go through PRs).Otherwise we would need to create a fork of the repo (which is more difficult to discover and sometime unclear to users which repo to use).
The text was updated successfully, but these errors were encountered: