[Instrumentation.EFCore] Support together with Instrumentation.SqlClient

[zivaninstana]

Fixes #1764
Design discussion issue #

Changes

#1764 (comment)

Merge requirement checklist

• CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
• Unit tests added/updated
• Appropriate CHANGELOG.md files updated for non-trivial changes
• Changes in public API reviewed (if applicable)

[codecov]

Codecov Report

Attention: Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.

Project coverage is 56.77%. Comparing base (71655ce) to head (0e33045).
Report is 600 commits behind head on main.

Files with missing lines	Patch %	Lines
...mplementation/EntityFrameworkDiagnosticListener.cs	87.50%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #2280       +/-   ##
===========================================
- Coverage   73.91%   56.77%   -17.15%     
===========================================
  Files         267        5      -262     
  Lines        9615      192     -9423     
===========================================
- Hits         7107      109     -6998     
+ Misses       2508       83     -2425     

Flag	Coverage Δ
unittests-Instrumentation.EntityFrameworkCore	56.77% <87.50%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...mplementation/EntityFrameworkDiagnosticListener.cs	52.63% <87.50%> (+0.25%)	⬆️

... and 263 files with indirect coverage changes

[Kielek]

Please add CHANGELOG.md entry

[CodeBlanch]

Please add CHANGELOG.md entry

A test would be great as well! Is it possible to model a test for this fix?

[zivaninstana]

Please add CHANGELOG.md entry

A test would be great as well! Is it possible to model a test for this fix?

I will see what can be implemented 👍

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 7 days.