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

remove vsc import step from README #167

Merged
merged 3 commits into from
Aug 16, 2023

Conversation

moriarty
Copy link
Collaborator

This vcs step is no longer required because the robotiq_description package has been released.

This vcs step causes confusion because it pulls in all of the robotiq repository and part of it depends on serial which hasn't been released for ROS 2.

See this issue:
PickNikRobotics/ros2_robotiq_gripper#21

This vcs step is no longer required because the robotiq_description
package has been released.

This vcs step causes confusion because it pulls in all of the robotiq
repository and part of it depends on serial which hasn't been released
for ROS 2.

See this issue:
PickNikRobotics/ros2_robotiq_gripper#21

Signed-off-by: Alex Moriarty <[email protected]>
Most users should be happy with the binary install as long as it is 0.2.2 or higher

Signed-off-by: Alex Moriarty <[email protected]>
Signed-off-by: Alex Moriarty <[email protected]>
@moriarty
Copy link
Collaborator Author

This PR also fixes a TODO which I was suppose to update it once the packages have been released.

They are available in Iron, but Humble and Rolling they are released but they just need a sync.

I added a link to https://docs.ros.org/en/rolling/Installation/Testing.html in the readme because if you install ros-testing then you don't need to wait for sync and get the latest fixes (might also get the latest bugs but more often you get fixes)

@moriarty moriarty enabled auto-merge (squash) August 15, 2023 19:43
@moriarty moriarty merged commit a2f018a into main Aug 16, 2023
10 checks passed
@moriarty moriarty deleted the fixup-readme-remove-vcs-import-step branch August 16, 2023 13: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.

2 participants