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

Style: Add .clangd config #100828

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Repiteo
Copy link
Contributor

@Repiteo Repiteo commented Dec 26, 2024

After testing clangd once I saw how easily the tool finds unused headers1, the intellisense/performance benefits it provided were immediately apparant. However, to be fully suited for the repo, a few adjustments needed to be made via configuration. This PR achieves exactly that, adding a minimal .clangd to allow the repo to "properly" utilize this tool. I fully expect this file to expand over time, as this is very much a jumping-off point. Also extends .editorconfig to recognize the clang config files as yaml.

Footnotes

  1. https://github.com/godotengine/godot/pull/100634#issuecomment-2556801462

@Repiteo Repiteo added this to the 4.4 milestone Dec 26, 2024
@Repiteo Repiteo requested a review from a team as a code owner December 26, 2024 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant