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

cpp pack contains both clangd and clang-format #612

Closed
3 tasks done
dpriedel opened this issue Oct 18, 2023 · 4 comments · Fixed by #619
Closed
3 tasks done

cpp pack contains both clangd and clang-format #612

dpriedel opened this issue Oct 18, 2023 · 4 comments · Fixed by #619
Labels
good first issue Good for newcomers

Comments

@dpriedel
Copy link

Checklist

  • I have searched through the AstroNvim docs
  • I have searched through the existing issues of AstroNvim
  • I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)

0.9.4

Operating system/version

Manjaro Linux

Terminal/GUI

tilix

Describe the bug

The cpp pack contains both clangd and clang-format. clangd includes the clang-format functionality https://clangd.llvm.org/features.html so having both seems unnecessary.

Steps to Reproduce

  1. install cpp pack
  2. edit cpp file
  3. both clangd and clang-format are enabled.

Expected behavior

Only clangd is required to have lsp and code formatting

Screenshots

No response

Additional Context

No response

@dpriedel dpriedel added the bug Something isn't working label Oct 18, 2023
@Uzaaft Uzaaft added the good first issue Good for newcomers label Oct 18, 2023
@Uzaaft
Copy link
Member

Uzaaft commented Oct 18, 2023

Feel free to open a PR. :)

@Uzaaft Uzaaft removed the bug Something isn't working label Oct 18, 2023
@dpriedel
Copy link
Author

dpriedel commented Oct 18, 2023 via email

@dpriedel
Copy link
Author

Thanks!!
I'm afraid I got busy and didn't have time to try to do this.

@Uzaaft
Copy link
Member

Uzaaft commented Oct 23, 2023

No worries. I fixed it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants