Skip to content

Fix regression in API impacting Codespaces - #2978

Merged
datho7561 merged 1 commit into
redhat-developer:masterfrom
datho7561:2968-return-api-lightweight
Mar 2, 2023
Merged

datho7561 merged 1 commit into
redhat-developer:masterfrom
datho7561:2968-return-api-lightweight

Conversation

@datho7561

Copy link
Copy Markdown
Contributor

The API wouldn't report lightweight mode correctly when it was forced to start in lightweight due to vscode-java detecting it was being run in Codespaces. This broke the mechanism to switch to standard mode in Codespaces.

Fixes #2968

Signed-off-by: David Thompson davthomp@redhat.com

The API wouldn't report lightweight mode correctly when it was forced to
start in lightweight due to vscode-java detecting it was being run in
Codespaces.
This broke the mechanism to switch to standard mode in Codespaces.

Fixes redhat-developer#2968

Signed-off-by: David Thompson <davthomp@redhat.com>
@datho7561

Copy link
Copy Markdown
Contributor Author

To test this PR:

  1. Make sure you are in Codespaces
  2. Use the default settings, which should mean that vscode-java forces LightWeight mode
  3. Make sure that the java language server workspace is clean (I wasn't able to reproduce the issue if I had successfully opened the workspace in standard mode already)
  4. Every time Codespaces restarts, it will uninstall all the .vsix that were manually installed from your computer, so you will need to install the .vsix at each launch

@datho7561
datho7561 requested a review from rgrunber March 2, 2023 18:40

@rgrunber rgrunber left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks like this is just restoring tbe part of https://github.com/redhat-developer/vscode-java/pull/2905/files#diff-04bba6a35cad1c794cbbe677678a51de13441b7a6ee8592b7b50be1f05c6f626L375-L378 that updated the server mode.
Feel free to merge when ready.

@datho7561
datho7561 merged commit e3778eb into redhat-developer:master Mar 2, 2023
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.

v1.15.x causes Java to be non-functional in GitHub Codespaces

2 participants