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

Peripheral Inspector extension is not available in remote ssh state #30

Open
lyflyaway opened this issue Nov 15, 2024 · 3 comments · May be fixed by #31
Open

Peripheral Inspector extension is not available in remote ssh state #30

lyflyaway opened this issue Nov 15, 2024 · 3 comments · May be fixed by #31

Comments

@lyflyaway
Copy link

Type: Feature Request

Hello, as the following picture, in the Visual Studio Code remote ssh state, Peripheral Inspector extension is not available. Could you add this feature? Thank you!

Peripheral Inspector

@thegecko
Copy link
Contributor

This is going to be a similar situation as the memory inspector: eclipse-cdt-cloud/vscode-memory-inspector#150

@thegecko thegecko linked a pull request Nov 15, 2024 that will close this issue
@lyflyaway
Copy link
Author

Hello thegecko,

After I add "workspace" in the package.json, the Peripheral Inspector extension state is change, as the following picture, is no longer gray, but it indicates Reload Window.

Even though I click Reload Window, it still doesn't work and indicates Reload Window again. Could you help me take a look at it? Thank you!

Peripheral Inspector state
Peripheral Inspector is not loaded

@thegecko
Copy link
Contributor

The error suggests there is a hard dependency between extensions. I would recommend you don't have a concrete coupling like this as it causes a lot of issues.

The approach I would recommend:

  • If you want to install extensions in a bundle - have your main one act as an extensionPack (in your case this looks like renesas debug is the main one)
  • If you rely on functionality in another extension, check it is installed first (vscode.extensions.all) and fail gracefully. If its there you can activate it to get the Api or call known commands from it

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 a pull request may close this issue.

2 participants