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

clangd and clang-tidy initial support #466

Merged
merged 2 commits into from
Jul 18, 2022
Merged

clangd and clang-tidy initial support #466

merged 2 commits into from
Jul 18, 2022

Conversation

NikitaNikolaenko
Copy link
Contributor

Allows clangd LSP to work in VS Code (or any other IDE/Code Editor that has some kind of a support for clangd).

The output of the added rule can also be used in clang-tidy (which is how #465 PR was created).

The README in https://github.com/hedronvision/bazel-compile-commands-extractor was especially helpful in the process of setting everything up.

Copy link
Contributor

@CodingCanuck CodingCanuck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! :D

.clang-tidy Show resolved Hide resolved
.clang-tidy Outdated Show resolved Hide resolved
.clang-tidy Outdated Show resolved Hide resolved
.clangd Show resolved Hide resolved
WORKSPACE.bazel Outdated Show resolved Hide resolved
@NikitaNikolaenko NikitaNikolaenko force-pushed the folming.clangd branch 6 times, most recently from a7e11e9 to 6423309 Compare July 18, 2022 09:52
@aviator-app
Copy link

aviator-app bot commented Jul 18, 2022

Aviator status

Aviator will automatically update this comment as the status of the PR changes.

This PR was merged using Aviator.

@aviator-app
Copy link

aviator-app bot commented Jul 18, 2022

Branch up-to-date, waiting for checks.

1 similar comment
@aviator-app
Copy link

aviator-app bot commented Jul 18, 2022

Branch up-to-date, waiting for checks.

@aviator-app aviator-app bot merged commit 5a04a50 into main Jul 18, 2022
@aviator-app aviator-app bot deleted the folming.clangd branch July 18, 2022 10:17
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.

3 participants