googleapis / googleapis/google-cloud-cpp

Enable checks that require code cleanup in .clang-tidy

Đang mở
#14,162 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
type: cleanup
Ngôn ngữ chính
C++
Star
659
Fork
462
Merge trung bình
1 ngày 2 giờ
Pull request đã merge (30 ngày)
89

Mô tả

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.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

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.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
cpp
Lĩnh vực
tooling
Loại issue
Tái cấu trúc
Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.