notskm / vscode-clang-tidy

MIT License
48 stars 26 forks source link

Add quickfix support to vscode-lang-tidy #46

Closed dmeister closed 3 years ago

dmeister commented 4 years ago

This patch adds quickfix support so that the many clang-tidy checks with fixes can automatically be applied to the code. This is related to issue #42 .

Unfortunatelly, the patch also contains some refactoring, which helped me to find my way around the code. If there is a general interest in merging this patch, I can, of course, remove these refactorings.

mfiumara commented 4 years ago

I'd be interested in this feature, looks like it needs some linting for it to pass CI.

benvanik commented 4 years ago

also interested in this!

notskm commented 3 years ago

Looks good! Thank you!

Can you please just run npm run format:fix and push the changes?

dmeister commented 3 years ago

Hello,

I just today saw this update. Next time, I know to run format:fix. Thank you, Dirk