Skip to content

clangd and clang-tidy initial support#466

Merged
aviator-app[bot] merged 2 commits into
mainfrom
folming.clangd
Jul 18, 2022
Merged

clangd and clang-tidy initial support#466
aviator-app[bot] merged 2 commits into
mainfrom
folming.clangd

Conversation

@NikitaNikolaenko

Copy link
Copy Markdown
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.

@CodingCanuck CodingCanuck left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Awesome! :D

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

aviator-app Bot commented Jul 18, 2022

Copy link
Copy Markdown

Aviator status

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

This PR was merged using Aviator.

@aviator-app

aviator-app Bot commented Jul 18, 2022

Copy link
Copy Markdown

Branch up-to-date, waiting for checks.

1 similar comment
@aviator-app

aviator-app Bot commented Jul 18, 2022

Copy link
Copy Markdown

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