googleapis / googleapis/google-cloud-cpp
Enable checks that require code cleanup in .clang-tidy
- Lenguaje dominante
- C++
- Estrellas
- 659
- Forks
- 462
- Merge medio
- 1 d 2 h
- PR fusionados (30 d)
- 89
Descripción
To enable it, remove it from the list of committed checks.
These checks are nice to have, but are current code does not conform to them. We need to cleanup the existing code first.
Checks to enable:
- [ ] bugprone-unused-return-value
- [ ] bugprone-inc-dec-in-condition
- [ ] bugprone-unused-local-non-trivial-variable
- [ ] misc-include-cleaner: too many *true* positives. We need to clean up the code, but we should do that over time, maybe per-directory, instead of blocking the adoption of Clang 17 until it is all fixed.
Guía de contribución
Línea de trabajo
Start by reviewing the committed-check list in .clang-tidy and the four checks named in the issue. Inspect existing violations across the codebase, clean them up incrementally, and consider directory-level progress for misc-include-cleaner; done means the listed checks can be enabled without the current code violations.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- tooling
- Tipo de issue
- Refactorización
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100