Skip to content

Conversation

@motty-mio2
Copy link

@motty-mio2 motty-mio2 commented Jan 16, 2026

Description

VSCode ROS Extension (maintained By ms-iot has been archived.
So I replace it with a successor extension.

Type of change

  • New feature (non-breaking change which adds functionality)

Screenshots

Please attach before and after screenshots of the change if applicable.

Checklist

  • I have read and understood the contribution guidelines
  • I have run the pre-commit checks with ./isaaclab.sh --format
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the changelog and the corresponding version in the extension's config/extension.toml file
  • I have added my name to the CONTRIBUTORS.md or my name already exists there

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 16, 2026

Greptile Summary

Replaces the archived ms-iot.vscode-ros extension with its maintained successor Ranch-Hand-Robotics.rde-pack in VSCode recommendations.

  • Updated main .vscode/extensions.json to use Ranch-Hand-Robotics.rde-pack instead of ms-iot.vscode-ros
  • Updated template configuration at tools/template/templates/external/.vscode/extensions.json to match
  • Added contributor name to CONTRIBUTORS.md

The old ms-iot.vscode-ros extension has been archived and is no longer maintained. The new Ranch-Hand-Robotics.rde-pack extension is the actively maintained successor that provides ROS 2 development support. This change ensures developers using VSCode will be recommended a supported extension for working with ROS-related files (URDF, launch files) in the IsaacLab codebase.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The changes are straightforward and low-risk: replacing an archived VSCode extension recommendation with its maintained successor. Both extension recommendation files were consistently updated, and the contributor was properly added to CONTRIBUTORS.md. No code logic or functionality is affected - only developer tooling recommendations.
  • No files require special attention

Important Files Changed

Filename Overview
.vscode/extensions.json Replaced archived ms-iot.vscode-ros with maintained Ranch-Hand-Robotics.rde-pack extension
tools/template/templates/external/.vscode/extensions.json Updated template to use the new maintained ROS extension
CONTRIBUTORS.md Added contributor name in alphabetical order

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant VSCode as VSCode Editor
    participant OldExt as ms-iot.vscode-ros<br/>(archived)
    participant NewExt as Ranch-Hand-Robotics.rde-pack<br/>(maintained)
    participant Repo as IsaacLab Repository

    Note over Dev,Repo: Before Changes
    Dev->>Repo: Open project in VSCode
    Repo->>VSCode: Load .vscode/extensions.json
    VSCode->>OldExt: Recommend ms-iot.vscode-ros
    Note over OldExt: Extension is archived<br/>and no longer maintained
    
    Note over Dev,Repo: After Changes
    Dev->>Repo: Open project in VSCode
    Repo->>VSCode: Load updated .vscode/extensions.json
    VSCode->>NewExt: Recommend Ranch-Hand-Robotics.rde-pack
    NewExt->>VSCode: Provide ROS 2 support
    Note over NewExt: Active successor extension<br/>with ROS 2 support
    Dev->>VSCode: Work with ROS files (URDF, launch files)
    NewExt->>Dev: ROS development features available
Loading

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 16, 2026

Greptile found no issues!

From now on, if a review finishes and we haven't found any issues, we will not post anything, but you can confirm that we reviewed your changes in the status check section.

This feature can be toggled off in your Code Review Settings by deselecting "Create a status check for each PR".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant