[Code Health] Clang Tidy cleanup, Part 2

[msiddhu]

Contributes to #2053

Changes

In continuation to Cleanup 1, this changes reduces a lot of clang-tidy warnings and removed some checks that are not relevant.

For significant contributions please make sure you have completed the following items:

• CHANGELOG.md updated for non-trivial changes
• Unit tests have been added
• Changes in public API reviewed

[codecov]

Codecov Report

Attention: Patch coverage is 47.05882% with 9 lines in your changes missing coverage. Please review.

Project coverage is 87.63%. Comparing base (497eaf4) to head (0e144cc).
Report is 120 commits behind head on main.

Files	Patch %	Lines
...k/src/metrics/aggregation/lastvalue_aggregation.cc	40.00%	6 Missing ⚠️
sdk/src/metrics/aggregation/sum_aggregation.cc	0.00%	2 Missing ⚠️
sdk/src/logs/logger_provider.cc	50.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3038      +/-   ##
==========================================
+ Coverage   87.12%   87.63%   +0.52%     
==========================================
  Files         200      190      -10     
  Lines        6109     5868     -241     
==========================================
- Hits         5322     5142     -180     
+ Misses        787      726      -61     

Files	Coverage Δ
...include/opentelemetry/sdk/common/circular_buffer.h	100.00% <ø> (ø)
sdk/src/common/base64.cc	82.93% <100.00%> (+0.53%)	⬆️
sdk/src/metrics/state/sync_metric_storage.cc	100.00% <100.00%> (ø)
sdk/src/trace/batch_span_processor.cc	94.12% <100.00%> (+1.70%)	⬆️
sdk/src/logs/logger_provider.cc	90.91% <50.00%> (ø)
sdk/src/metrics/aggregation/sum_aggregation.cc	75.81% <0.00%> (ø)
...k/src/metrics/aggregation/lastvalue_aggregation.cc	65.63% <40.00%> (-1.04%)	⬇️

... and 116 files with indirect coverage changes

[marcalff]

@msiddhu

This set of changes looks good already. Please tag as ready for review, so this part can be merged.

[marcalff]

LGTM, thanks for the cleanup.