googleapis / googleapis/google-cloud-cpp
Enable checks that require code cleanup in .clang-tidy
- 主要言語
- C++
- スター
- 659
- フォーク
- 462
- 平均マージ
- 1日 2時間
- マージ済み PR(30日)
- 89
説明
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.
コントリビューションガイド
調査の方向性
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.
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp
- 領域
- tooling
- issue の種類
- リファクタリング
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100