Skip to content

Delete Java LS Configuration#4160

Merged
rgrunber merged 1 commit intoredhat-developer:mainfrom
snjeza:issue-4157
Sep 17, 2025
Merged

Delete Java LS Configuration#4160
rgrunber merged 1 commit intoredhat-developer:mainfrom
snjeza:issue-4157

Conversation

@snjeza
Copy link
Copy Markdown
Collaborator

@snjeza snjeza commented Sep 16, 2025

Fixes #4157

Comment thread src/utils.ts
}

export function cleanJavaLSConfiguration(context: ExtensionContext) {
const globalStoragePath = context.globalStorageUri?.fsPath; // .../Code/User/globalStorage/redhat.java
Copy link
Copy Markdown
Member

@rgrunber rgrunber Sep 16, 2025

Choose a reason for hiding this comment

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

Just to confirm, this folder contains all vscode-java cached startup info for each version. If a user updates to a new pre-release, or regular release, the previous cache is here. (ie. $HOME/.config/Code/User/globalStorage/redhat.java/${version}/config_linux/org.eclipse.osgi).

Are we ok with removing these as well ? It could be beneficial in terms of keeping the size of globalStorage low.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

The PR removes whole $HOME/.config/Code/User/globalStorage/redhat.java/

Copy link
Copy Markdown
Member

@rgrunber rgrunber left a comment

Choose a reason for hiding this comment

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

I would just make the commit message body wrap at around 70 lines.

- delete the Java LS configuration (<globalStorage>/redhat.java) when callin g Clean Java Language Server Workspace

Otherwise, change seems fine.

- delete the Java LS configuration (<globalStorage>/redhat.java)
when calling Clean Java Language Server Workspace
@snjeza
Copy link
Copy Markdown
Collaborator Author

snjeza commented Sep 16, 2025

I would just make the commit message body wrap at around 70 lines.

Fixed.

@rgrunber rgrunber merged commit 1bb0a99 into redhat-developer:main Sep 17, 2025
2 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.

Delete Java LS configuration

2 participants