Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VSCode: Prevent unnecessary Java parsing when Java support if off. #5887

Merged
merged 1 commit into from
Apr 27, 2023

Conversation

dbalek
Copy link
Contributor

@dbalek dbalek commented Apr 25, 2023

Modified to prevent unnecessary parsing of Java sources invoked by Java editor features when Java support if off in Language Server settings.

@dbalek dbalek added LSP [ci] enable Language Server Protocol tests VSCode Extension [ci] enable VSCode Extension tests labels Apr 25, 2023
@dbalek dbalek added this to the NB19 milestone Apr 25, 2023
@dbalek dbalek requested review from MartinBalin and sdedic April 25, 2023 15:42
@MartinBalin
Copy link
Contributor

I assume this is also safe for NB 18.

@dbalek dbalek merged commit 40a0ecf into apache:master Apr 27, 2023
@dbalek dbalek deleted the dbalek/vscode-java-off branch April 27, 2023 07:51
@neilcsmith-net
Copy link
Member

I assume this is also safe for NB 18.

I assume it was decided not to add to NB18?

@MartinBalin
Copy link
Contributor

I assume this is also safe for NB 18.

I assume it was decided not to add to NB18?

Yes, not to add to NB18.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LSP [ci] enable Language Server Protocol tests VSCode Extension [ci] enable VSCode Extension tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants