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

VSX: Add 'Local' PluginType #9247

Closed
wants to merge 1 commit into from

Conversation

seantan22
Copy link
Contributor

What it does

How to test

  1. Open the extensions-view.
  2. Install an extension using the Install From VSIX... command.
  3. Confirm that the extension is of PluginType Local.

Review checklist

Reminder for reviewers

Signed-off-by: seantan22 sean.a.tan@ericsson.com

What it does
- Adds `Local` as a `PluginType` to refer to plugins that are installed
  locally (e.g. uploaded using `Install From VSIX...`)

How to test
1. Open the extensions-view.
2. Install an extension using the `Install From VSIX...` command.
3. Confirm that the extension is of PluginType `Local`.

Signed-off-by: seantan22 <sean.a.tan@ericsson.com>
@vince-fugnitto vince-fugnitto added plug-in system issues related to the plug-in system vsx-registry Issues related to Open VSX Registry Integration labels Mar 24, 2021
Copy link
Contributor

@DucNgn DucNgn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@vince-fugnitto
Copy link
Member

@benoitf any objections to introducing the local plugin-type which helps differentiate between system installed, user installed and local (uploaded) extensions?

@benoitf
Copy link
Contributor

benoitf commented Mar 29, 2021

hello, to me it looks like it's still a User scope as it's installed by the user (not a built-in)

it seems we're making a specific case around openvsx registry.

@paul-marechal
Copy link
Member

In this case what's the actual meaning for plugin type? If it is solely to differentiate between builtins and non-builtins, should we make a new field pluginOrigin to denote if the plugin came from openvsx or --plugins dir? Do we need to differentiate those plugins in the first place?

@benoitf
Copy link
Contributor

benoitf commented Mar 29, 2021

@marechal-p yes I think you're more interested in adding the original source
For example let say you could tomorrow use VS Code marketplace (or you use another marketplace, local should not be tied to 'open-vsix' but openvsx extension is interested in getting all plugins having that flag (and be with user or system scope)

For example you might still be interested to know that a 'default extension' has still updates, etc. (so it's a System plugin type but you may want to check something around it)

@vince-fugnitto
Copy link
Member

Closing in favor of an upcoming pull-request which handles local plugins directly in @theia/vsx-registry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plug-in system issues related to the plug-in system vsx-registry Issues related to Open VSX Registry Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants