[API] DO not allow unsafe Logger::EmitLogRecord

[owent]

Fixes #2666

Changes

• Do not allow unsafe Logger::EmitLogRecord

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

[marcalff]

Suggested change

	"unique_ptr<LogRecord> is not allowed");
	"unique_ptr<LogRecord> is not allowed, use std::move()");

Also, consider adding an assert for LogRecord*

[owent]

Thanks, it's done

[marcalff]

Thanks for the fix, see suggestion.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.70%. Comparing base (497eaf4) to head (eea093d).
Report is 66 commits behind head on main.

❗ Current head eea093d differs from pull request most recent head 1f23592

Please upload reports for the commit 1f23592 to get more accurate results.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2673      +/-   ##
==========================================
+ Coverage   87.12%   87.70%   +0.58%     
==========================================
  Files         200      190      -10     
  Lines        6109     5852     -257     
==========================================
- Hits         5322     5132     -190     
+ Misses        787      720      -67     

Files	Coverage Δ
...pi/include/opentelemetry/logs/logger_type_traits.h	100.00% <ø> (+16.00%)	⬆️

... and 56 files with indirect coverage changes

[marcalff]

LGTM, thanks for the fix.