Skip to content

Conversation

@RomanNikitenko
Copy link
Member

@RomanNikitenko RomanNikitenko commented Apr 3, 2025

What does this pull request change?

Adds instructions how to disable Install from VSIX command for the Code-OSS editor.

image image image

What issues does this pull request fix or reference?

It was done for the https://issues.redhat.com/browse/CRW-8313
PR in the Che-Code: che-incubator/che-code#537

Specify the version of the product this pull request applies to

Che-Code - 7.100
DevSpaces-Code - 3.20

Pull Request checklist

The author and the reviewers validate the content of this pull request with the following checklist, in addition to the automated tests.

  • Any procedure:
    • Successfully tested.
  • Any page or link rename:
    • The page contains a redirection for the previous URL.
    • Propagate the URL change in:
  • Builds on Eclipse Che hosted by Red Hat.
  • the Validate language on files added or modified step reports no vale warnings.

@github-actions
Copy link

github-actions bot commented Apr 3, 2025

Click here to review and test in web IDE: Contribute

@github-actions
Copy link

github-actions bot commented Apr 3, 2025

🎊 Navigate the preview: https://67efa3f99f5e4ee56179aedc--eclipse-che-docs-pr.netlify.app 🎊

@RomanNikitenko RomanNikitenko marked this pull request as ready for review April 3, 2025 11:20
@RomanNikitenko RomanNikitenko requested review from a team and deerskindoll as code owners April 3, 2025 11:20
@RomanNikitenko RomanNikitenko requested a review from amisevsk April 3, 2025 11:20
Copy link
Contributor

@deerskindoll deerskindoll left a comment

Choose a reason for hiding this comment

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

a couple of tweaks

RomanNikitenko and others added 2 commits April 3, 2025 14:39
…icrosoft-visual-studio-code.adoc

Co-authored-by: Jana Vrbkova <[email protected]>
…icrosoft-visual-studio-code.adoc

Co-authored-by: Jana Vrbkova <[email protected]>
Copy link
Contributor

@deerskindoll deerskindoll left a comment

Choose a reason for hiding this comment

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

lgtm

@RomanNikitenko
Copy link
Member Author

Thank you both for reviewing my PR!

@deerskindoll
I see that the build failed
is it related to my changes?
or
it's not and I can merge my PR?

@RomanNikitenko RomanNikitenko merged commit edea702 into eclipse-che:main Apr 4, 2025
6 checks passed
@RomanNikitenko RomanNikitenko deleted the install-from-vsix-disabled branch April 4, 2025 09:06
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 this pull request may close these issues.

3 participants