Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Alignment with 1.100.3 version of VS Code#547

Merged
RomanNikitenko merged 1428 commits into
che-incubator:mainfrom
RomanNikitenko:alignment-with-upstream-1-100
Jul 11, 2025
Merged

Alignment with 1.100.3 version of VS Code#547
RomanNikitenko merged 1428 commits into
che-incubator:mainfrom
RomanNikitenko:alignment-with-upstream-1-100

Conversation

@RomanNikitenko
Copy link
Copy Markdown
Collaborator

@RomanNikitenko RomanNikitenko commented May 15, 2025

Note: I created a separate issue for a bug I detected within testing, please see eclipse-che/che#23486

What does this PR do?

What issues does this PR fix?

How to test this PR?

It makes sense to test both: ubi8 and ubi9-based assemblies.

Go to the dashboard => Create Workspace page => set quay.io/che-incubator-pull-requests/che-code:pr-547-amd64 to the Editor Image field and select a sample to start a workspace
or click
https://che-dogfooding.apps.che-dev.x6e0.p1.openshiftapps.com/dashboard/#/load-factory?editor-image=quay.io/che-incubator-pull-requests/che-code:pr-547-amd64&url=https://github.com/che-samples/web-nodejs-sample

1. Test any functionality, I tested the following:

  • open a file
  • create a default terminal
  • create a terminal in a container
  • instal an extension
  • uninstall an extension
  • run a devfile command
  • run a command from the Command Palette
  • Git Clone flow from the Command Palette

2. Restart workspace from a local devfile:

  • start workspace with a devfile (there is a reference above for the web-nodejs-sample)
  • open devfile.yaml file
  • provide a change (I changed a command)
  • F1 => Restart Workspace from a Local Devfile
  • check that the change was applied (I run the command and tested that my change was applied)

3. Connect to Ansible lightspeed server form Ansible extension view:

I also tested the use case described in #532 to avoid a regression
image

4. Copilot-related items should be absent by default in the Command center area

I tested the use case described in #549

5. Hot key paste text to the 'Search settings' field

I tested that current PR changes fix the use case described in eclipse-che/che#23459

6. Released version of an extension should be installed

I detected a bug for the case when an extension contains another extension as a dependency, please see eclipse-che/che#23486

Does this PR contain changes that override default upstream Code-OSS behavior?

  • the PR contains changes in the code folder (you can skip it if your changes are placed in a che extension )
  • the corresponding items were added to the CHANGELOG.md file
  • rules for automatic git rebase were added to the .rebase folder

deepak1556 and others added 30 commits April 28, 2025 07:02
Previously I split these up by js and ts. This ends up creating too many group and having the js and ts settings too far from each other

This PR changes the settings to be grouped only by feature, not language
- updated font sizing for new walkthrough layout
* add replace listeners -- wip

* guess what? typescript isn't an HDL. order matters smh
@RomanNikitenko RomanNikitenko force-pushed the alignment-with-upstream-1-100 branch from bcbcc29 to 3999823 Compare July 4, 2025 08:12
@RomanNikitenko RomanNikitenko force-pushed the alignment-with-upstream-1-100 branch from 3ebbdf2 to dcf65f5 Compare July 4, 2025 10:00
@RomanNikitenko RomanNikitenko force-pushed the alignment-with-upstream-1-100 branch 2 times, most recently from d0e806a to ad43c0b Compare July 4, 2025 14:56
@RomanNikitenko RomanNikitenko force-pushed the alignment-with-upstream-1-100 branch from ad43c0b to 16ee942 Compare July 4, 2025 15:00
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 4, 2025

1 similar comment
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 4, 2025

@RomanNikitenko RomanNikitenko force-pushed the alignment-with-upstream-1-100 branch from 16ee942 to c5d7ba3 Compare July 4, 2025 15:42
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 4, 2025

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 5, 2025

@RomanNikitenko RomanNikitenko changed the title Alignment with upstream 1.100 Alignment with upstream 1.100.3 Jul 8, 2025
@RomanNikitenko RomanNikitenko changed the title Alignment with upstream 1.100.3 Alignment with 1.100.3 version of VS Code Jul 8, 2025
@RomanNikitenko RomanNikitenko marked this pull request as ready for review July 9, 2025 11:03
@github-actions
Copy link
Copy Markdown

@ibuziuk ibuziuk self-requested a review July 11, 2025 11:20
@RomanNikitenko RomanNikitenko merged commit 58daa02 into che-incubator:main Jul 11, 2025
12 of 29 checks passed
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.