googleapis / googleapis/google-cloud-cpp

Enable checks that require code cleanup in .clang-tidy

Ouverte
#14,162 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
type: cleanup
Langage dominant
C++
Étoiles
659
Forks
462
Merge moyen
1 j 2 h
PR mergées (30 j)
89

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
tooling
Type d'issue
Refactorisation
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.