Skip to content

Remove moveit2_tutorials.repos entries now available in Rolling - #1071

Merged
130s merged 3 commits into
moveit:mainfrom
nbbrooks:fix-moveit2-tutorials-ci
Aug 19, 2026
Merged

Remove moveit2_tutorials.repos entries now available in Rolling#1071
130s merged 3 commits into
moveit:mainfrom
nbbrooks:fix-moveit2-tutorials-ci

Conversation

@nbbrooks

@nbbrooks nbbrooks commented Oct 12, 2025

Copy link
Copy Markdown
Contributor

Description

This plus PickNikRobotics/rviz_visual_tools#284 will fix moveit2_tutorial rolling CI job because the ros2_kortex source checkout was trying to use the gripper_controllers removed form Kilted and Rolling and not parallel_gripper_controller (see Kinovarobotics/ros2_kortex#294)

Checklist

  • Required by CI: Code is auto formatted using clang-format
  • While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers

Summary by CodeRabbit

  • Chores
    • Removed obsolete repository dependencies from the MoveIt 2 tutorials configuration.
    • Simplified setup by eliminating unused external repositories.

…were available

This will fix moveit2_tutorial rolling CI job because the ros2_kortex source checkout was trying to use the gripper_controllers removed form Kilted and Rolling and not parallel_gripper_controller (see Kinovarobotics/ros2_kortex#294)
@nbbrooks nbbrooks mentioned this pull request Nov 10, 2025
2 tasks
@nbbrooks

Copy link
Copy Markdown
Contributor Author

Failing due to ros2_robotiq_description and kortex_launch still depending on gripper_controllers in rolling (and kilted). Apparently noone uses rolling.

ros2_robotiq_description PR PickNikRobotics/ros2_robotiq_gripper#103
kortex_launch no PR yet

@sea-bass

sea-bass commented Dec 1, 2025

Copy link
Copy Markdown
Contributor

kortex_launch no PR yet

Kinovarobotics/ros2_kortex#324

@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: cf205198-f18e-423d-9a5f-7e439c31cd33

📥 Commits

Reviewing files that changed from the base of the PR and between a3fb6ae and bb8a2fb.

📒 Files selected for processing (1)
  • moveit2_tutorials.repos

📝 Walkthrough

Walkthrough

The repository manifest removes the ros2_kortex, ros2_robotiq_gripper, and serial dependencies.

Changes

Repository dependency cleanup

Layer / File(s) Summary
Remove repository entries
moveit2_tutorials.repos
The manifest no longer declares the ros2_kortex, ros2_robotiq_gripper, and serial repositories.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: andyze

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@130s
130s enabled auto-merge (squash) August 19, 2026 03:42
@130s
130s merged commit 17f6a5e into moveit:main Aug 19, 2026
6 of 9 checks passed
@github-project-automation github-project-automation Bot moved this to ✅ Done in MoveIt Aug 19, 2026
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.

4 participants