-
Notifications
You must be signed in to change notification settings - Fork 111
Conversation
Signed-off-by: Vitaliy Gulyy <[email protected]>
✅ E2E Happy path tests succeed 🎉 See Details
Tested with Eclipse Che Single User on K8S (minikube v1.1.1)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How I can verify these extensions are working?
What functionality do they contribute to Che Theia?
for vscode extension editing |
For search-result you can perform searching, and then open a file. Found text should be highlighted. |
I've the same error than Artem. (using the image built by the CI) |
Strange, but OK. I will try to update the extensions to their fresh versions. |
I tested it on Hosted Che and dogfooding instances. |
It's built by CI from this PR. |
We postponed with adding of those extensions as it's unclear for now why they are not working properly with upstream Theia. |
Signed-off-by: Vitaliy Gulyy [email protected]
What does this PR do?
Includes vscode.extension-editing and vscode.search-result extensions to list of default extensions.
Screenshot/screencast of this PR
What issues does this PR fix or reference?
eclipse-che/che#18282
How to test this PR?
Devfile to test changes (the extensions are injected to che-theia container as Che plugins)
PR Checklist
As the author of this Pull Request I made sure that:
What issues does this PR fix or reference
andHow to test this PR
completedReviewers
Reviewers, please comment how you tested the PR when approving it.
Happy Path Channel
HAPPY_PATH_CHANNEL=stable