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

LibertyTextDocument puts a lot of info in the log #266

Closed
turkeylurkey opened this issue Jan 19, 2024 · 2 comments · Fixed by #267
Closed

LibertyTextDocument puts a lot of info in the log #266

turkeylurkey opened this issue Jan 19, 2024 · 2 comments · Fixed by #267
Labels
enhancement New feature or request good first issue Good for newcomers LCLS Liberty Config Language Server

Comments

@turkeylurkey
Copy link
Member

As I edit my properties file in IntelliJ this language server prints the whole file to the log every time I make a change. This seems like too much output when there isn't an error.

Could you remove this log statement and only print something if there is an error or a warning is needed?

LibertyTextDocument: LOGGER.info(last.getText());

@turkeylurkey turkeylurkey added enhancement New feature or request good first issue Good for newcomers LCLS Liberty Config Language Server labels Jan 19, 2024
@cherylking
Copy link
Member

This seems like a mistake. It was probably for debugging and slipped in.

@turkeylurkey
Copy link
Member Author

At first I thought it was my own debugging log!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers LCLS Liberty Config Language Server
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants