Add hand_eye_calibration_sim to the config list

[fdavulcu]

[written by AI]

hand_eye_calibration_sim was hidden from moveit_pro run --list and moveit_pro configure until the hand-eye calibration Objectives shipped. They have, and the camera calibration guide now walks through this config, so the MOVEIT_PRO_IGNORE marker goes. The README sentence that described the marker goes with it.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: ca64ea31-6463-4dd2-ae5b-f72e0297e36c

📥 Commits

Reviewing files that changed from the base of the PR and between 0da511d and 0fd57b8.

📒 Files selected for processing (2)

• src/hand_eye_calibration_sim/MOVEIT_PRO_IGNORE
• src/hand_eye_calibration_sim/README.md

💤 Files with no reviewable changes (1)

• src/hand_eye_calibration_sim/MOVEIT_PRO_IGNORE

🚧 Files skipped from review as they are similar to previous changes (1)

• src/hand_eye_calibration_sim/README.md

Included review availability: 6 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 8 reviews per hour.

---

📝 Summary

Summary by CodeRabbit

• Documentation
  • Updated the hand-eye calibration simulator documentation by removing an outdated statement about application configuration.

Walkthrough

The package README no longer documents that MOVEIT_PRO_IGNORE keeps the package out of the app configuration list.

Changes

README documentation

Layer / File(s)	Summary
Remove configuration statement src/hand_eye_calibration_sim/README.md	Removes the statement about MOVEIT_PRO_IGNORE and package configuration.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 0fd57

The package is available to the launcher as intended, and the README no longer describes the removed exclusion behavior.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description check	✅ Passed	The pull request description directly matches the changes. It explains removal of the MOVEIT_PRO_IGNORE marker and the related README sentence because the hand-eye calibration Objectives have shipped.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Human Review Check	✅ Passed	The PR is low-risk under this check. The authoritative diff changes only two files in src/hand_eye_calibration_sim: it deletes the empty MOVEIT_PRO_IGNORE marker and removes one README sentence th…

---

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

[github-actions]

MoveIt Pro Example WS - Objectives Integration Test Report

• lab_sim
  • ✅ jazzy: view HTML report — 32 passed
• hangar_sim
  • ❌ jazzy: view HTML report — 7 failed, 11 passed

[github-actions]

MoveIt Pro Example WS - Objectives Integration Test Report

• lab_sim
  • ✅ jazzy: view HTML report — 32 passed
• hangar_sim
  • ❌ jazzy: view HTML report — 7 failed, 11 passed

[github-actions]

MoveIt Pro Example WS - Objectives Integration Test Report

• lab_sim
  • ✅ jazzy: view HTML report — 32 passed
• hangar_sim
  • ❌ jazzy: view HTML report — 7 failed, 11 passed