-
Notifications
You must be signed in to change notification settings - Fork 12
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
Inform user when clangd formatting fails #400
Comments
An additional thought on here. Can we/should we capture stderr from clangd to some user feedback? In the case of the clang format file being invalid clangd does show errors on stderr. The errors are the same ones that you are now checking with --check call, but there may be others in the future, and clang format files may exist in parent directories outside of Eclipse project that clangd uses too. Ideally clangd would not provide new formatting in the case of clang format errors. It seems vscode's extension suffers the same problem https://stackoverflow.com/a/78249405/2796832 |
Yes. I am not sure how to solve this issue right now. IMO there are several use cases here:
In this issue I would implement the second. |
Yes, that would be the best solution here. |
As a user I want to be informed when clangd could not format my source file because the .clang-format file contains an invalid configuration.
Background: clangd uses a fallback config when the .clang-format file is missing or contains an invalid configuration. This happens silently.
See also the PR #399 and issue #398
The text was updated successfully, but these errors were encountered: