Skip to content

Fix: Update lab_sim skip list for renamed Marker Visualization objective (v8.10 backport)#750

Closed
fdavulcu wants to merge 1 commit into
v8.10from
backport/skip-list-marker-visualization-v8.10
Closed

Fix: Update lab_sim skip list for renamed Marker Visualization objective (v8.10 backport)#750
fdavulcu wants to merge 1 commit into
v8.10from
backport/skip-list-marker-visualization-v8.10

Conversation

@fdavulcu

@fdavulcu fdavulcu commented Jun 30, 2026

Copy link
Copy Markdown

[written by AI]

Backports the skip-list fix in #462 to v8.10.

lab_sim's integration test skips objectives by name. moveit_pro renamed this objective to Marker Visualization Example, so the stale skip entry stopped matching; the objective then ran in headless CI, where its GetTextFromUser step has no /request_text_from_prompts server and failed. Correcting the name re-skips it.

Release notes

None

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@fdavulcu fdavulcu added this to the 8.10.8 milestone Jun 30, 2026
@coderabbitai

coderabbitai Bot commented Jun 30, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 947b51a0-d5a8-4e47-80e3-01b997c396c3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

@fdavulcu

Copy link
Copy Markdown
Author

[written by AI]

Superseded by #745, which now carries this same one-line lab_sim skip-list fix alongside the .gitattributes pre-commit fix. Both had to ship together: on v8.10, pre-commit and the integration test were mutually blocking, so neither could merge alone. Closing in favor of #745.

@fdavulcu fdavulcu closed this Jun 30, 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.

1 participant