-
Notifications
You must be signed in to change notification settings - Fork 99
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
arbotix_python for ROS Noetic #46
arbotix_python for ROS Noetic #46
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good - I think the one other thing we probably need to do is update the shebangs to to python3. Please also revert the couple of changelog/version things - those get automatically done as part of the release process.
arbotix_python/CHANGELOG.rst
Outdated
@@ -2,6 +2,13 @@ | |||
Changelog for package arbotix_python | |||
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | |||
|
|||
0.11.0 (2020-12-25) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please revert the changes to the CHANGELOG - this is automatically generated when I do the release.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
arbotix_python/package.xml
Outdated
<name>arbotix_python</name> | ||
<version>0.10.0</version> | ||
<version>0.11.0</version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please revert this one line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
Ouch - reverted the autostuff. I would leave the shebangs as is until somebody posts an issue :D. It worked with python 2, now works with python 3. If you prefer to explicitly name the version, give me a sign, I'll change it. |
Thanks for the PR and quick response to review comments. |
Hi Mike, yet another copy of arbotix_python working with ROS Noetic. Please give it a try. Tested arbotix_python and arbotix_terminal. I didn't touch other packages inside arbotix_ros besides the one mentioned above.