✅ Deploy Preview for opentelemetry-cpp-api-docs canceled.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.41%. Comparing base (021eb99) to head (8066432).
Report is 1 commits behind head on main.

@@           Coverage Diff           @@
##             main    #3285   +/-   ##
=======================================
  Coverage   89.41%   89.41%           
=======================================
  Files         207      207           
  Lines        6457     6457           
=======================================
  Hits         5773     5773           
  Misses        684      684           

@owent thanks for the PR. The event logger related changes look good in general. However, can we move the clang formatting related changes out of scope of this PR? we can have separate PR for that if needed.

Waiting on clarifications and/or cleanup for vscode file, before merging.

This change contains two independent parts:

• deprecation (abi v1) and removal (abi v2) of the event logger
• clang-format changes with compiler attributes macros

Approving this PR because both look ok.

See comments on vscode file.

@owent For future work, please separate independent changes in different PRs.

This helps with the git history, and helps when a backport is needed to an older branch for some fixes.

@lalitb @marcalff The format changes are included into this PR because there will be strange format style in some files of log's codes. They may cause some confuse.
I found that clang-format doesn't recognize our attribute macros as attributes but sometimes as class names. Including these format changes should help reduce confusion.