micro-ROS / micro-ROS/micro-ROS-demos
Disable fuchsia-related checks in clang-tidy
Aberta
@BorjaOuterelo já está trabalhando nisso.
Desde 16/1/2019.
enhancement
- Linguagem predominante
- C
- Estrelas
- 101
- Forks
- 35
- Merge médio
- 4d 16h
- PRs com merge (30d)
- 3
Descrição
Compiling the demos leads to a lot of warnings from clang-tidy, many of which are from the Fuchsia warning set, such as "fuchsia-default-arguments". Fuchsia has a very specific style guide which is not generally appropriate, e.g., calling functions with default arguments is valid C++ practice, but disallowed by Fuchsia.
I would suggest the following list instead of "*" as used currently:
bugprone-*,cert-*,clang-analyzer-*,cppcoreguidelines-*,google-*,hicpp-*,llvm-*,misc-*,modernize-*,performance-*,readability-*
Opinions?
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Avaliação
Esta issue ainda não foi avaliada.