Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Use theia-plugin-ext from node_modules. #464

Merged
merged 1 commit into from
Oct 3, 2019

Conversation

AndrienkoAleksandr
Copy link
Contributor

@AndrienkoAleksandr AndrienkoAleksandr commented Oct 2, 2019

What does this PR do?

Use theia-plugin-ext from node_modules. We need this because nexe tool doesn't resolve path with __dirname.

What issues does this PR fix or reference?

Needed for eclipse-che/che#13387

Signed-off-by: Oleksandr Andriienko oandriie@redhat.com

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@AndrienkoAleksandr
Copy link
Contributor Author

ci-build-check

@benoitf
Copy link
Contributor

benoitf commented Oct 2, 2019

I know why you're doing that but you could say in description that it's required as relative paths are not working when using nexe or pkg when trying to build single executable file, etc.

@artaleks9
Copy link

ci-build-check

@artaleks9
Copy link

ci-build

@che-bot
Copy link
Contributor

che-bot commented Oct 2, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

@AndrienkoAleksandr
Copy link
Contributor Author

crw-ci-test

@che-bot
Copy link
Contributor

che-bot commented Oct 2, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

@AndrienkoAleksandr
Copy link
Contributor Author

crw-ci-test

@che-bot
Copy link
Contributor

che-bot commented Oct 2, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

@AndrienkoAleksandr
Copy link
Contributor Author

@benoitf happy path passed, I think you waited on the result.

@benoitf
Copy link
Contributor

benoitf commented Oct 2, 2019

@AndrienkoAleksandr the issue is that it doesn't test extensions/eclipse-che-theia-plugin-remote/src/node/plugin-remote-backend-module.ts in the current happy path as it's using a predefined image for sidecar (and not the one that is built)

@AndrienkoAleksandr AndrienkoAleksandr merged commit 874bf6f into master Oct 3, 2019
@AndrienkoAleksandr AndrienkoAleksandr deleted the useTheiaPluginExtFromNodeModules branch October 3, 2019 09:23
vinokurig pushed a commit that referenced this pull request Apr 6, 2021
* Add buildkit-base dev plugin
* Change formatting
* Change containerPath on mountPath
* Allow to use any URI for plugin logo
* Set memory limitation to 1G
* Remove pattern to validate plugin logo image
* Return back pattern to validate plugin logo
* Update pattern to check plugin icon

Signed-off-by: Vitaliy Gulyy <vgulyy@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.

6 participants