Skip to content
This repository has been archived by the owner on Jun 20, 2018. It is now read-only.

Add support for vscode:extension/ resolver and http(s):// resolver #19

Merged
merged 2 commits into from
May 17, 2018

Conversation

benoitf
Copy link

@benoitf benoitf commented May 17, 2018

Allow to grab some vscode extension or download files hosted on a http(s) server

theia-vscode


export default new ContainerModule(bind => {
bind(PluginDeployerFileHandler).to(PluginVsCodeFileHandler).inSingletonScope();
bind(PluginDeployerDirectoryHandler).to(PluginVsCodeDirectoryHandler).inSingletonScope();
bind(PluginScanner).to(VsCodePluginScanner).inSingletonScope();
bind(PluginDeployerResolver).to(VsCodePluginDeployerResolver).inSingletonScope();

Copy link

Choose a reason for hiding this comment

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

Do we need this extra line?

Copy link
Author

Choose a reason for hiding this comment

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

fixed

console.log('filename is', filename);
console.log('dirname is', dirname);
/*
let filename = parsed.pathname pluginResolverContext.getOriginId().replace(/\W/g, '_');
Copy link

Choose a reason for hiding this comment

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

Is it needed? If so, format it properly, please.

Copy link
Author

Choose a reason for hiding this comment

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

good catch :-) it shouldn't have reached the PR

Copy link
Author

Choose a reason for hiding this comment

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

fixed

@benoitf benoitf merged commit 139abd7 into master May 17, 2018
@benoitf benoitf deleted the add-vscode-http-support branch May 17, 2018 13:45
@slemeur
Copy link

slemeur commented May 17, 2018

Impressive !

benoitf added a commit that referenced this pull request May 29, 2018
* Add support for vscode:extension/ resolver and http(s):// resolver
benoitf added a commit that referenced this pull request Jun 4, 2018
* Add support for vscode:extension/ resolver and http(s):// resolver
benoitf added a commit that referenced this pull request Jun 13, 2018
* Add support for vscode:extension/ resolver and http(s):// resolver
benoitf added a commit that referenced this pull request Jun 18, 2018
* Add support for vscode:extension/ resolver and http(s):// resolver
benoitf added a commit that referenced this pull request Jun 18, 2018
* Add support for vscode:extension/ resolver and http(s):// resolver
benoitf added a commit that referenced this pull request Jun 19, 2018
* Add support for vscode:extension/ resolver and http(s):// resolver
benoitf added a commit that referenced this pull request Jun 19, 2018
Signed-off-by: Florent BENOIT <fbenoit@redhat.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants