dense-analysis / dense-analysis/ale

clangtidy supports build_dir and cpp_clangtidy_options at the same time

Open
#3,118 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Vim Script
Stars
14k
Forks
1.5k
Avg merge
17h 49m
Merged PRs (30d)
1

Description

The `clang-tidy` command itself supports both `-p` and `--` at the same time. Checked in Debian version 9.0-49.1.

However the code snippet
https://github.com/dense-analysis/ale/blob/master/ale_linters/cpp/clangtidy.vim#L22-L26
```vim
if empty(l:build_dir)
let l:options = ale#Var(a:buffer, 'cpp_clangtidy_options')
let l:cflags = ale#c#GetCFlags(a:buffer, a:output)
let l:options .= !empty(l:options) ? ale#Pad(l:cflags) : l:cflags
endif
```
indicates that if `build_dir` provided, `cpp_clangtidy_options` will be ignored. Could you help support both at the same time?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.