[Instrumentation.AspNetCore] Nullable enabled

[Kielek]

Towards #894

Changes

[Instrumentation.AspNetCore] Nullable enabled
Move shared files to Shared folder

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

• [ ] Appropriate CHANGELOG.md updated for non-trivial changes
• [ ] Design discussion issue #
• [ ] Changes in public API reviewed

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.08%. Comparing base (71655ce) to head (7e5b561).
Report is 238 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1718      +/-   ##
==========================================
+ Coverage   73.91%   76.08%   +2.16%     
==========================================
  Files         267      297      +30     
  Lines        9615    11620    +2005     
==========================================
+ Hits         7107     8841    +1734     
- Misses       2508     2779     +271     

Flag	Coverage Δ
unittests-Exporter.Geneva	63.77% <ø> (?)
unittests-Exporter.OneCollector	89.72% <ø> (?)
unittests-Extensions	79.33% <ø> (?)
unittests-Instrumentation.AspNet	74.39% <ø> (?)
unittests-Instrumentation.AspNetCore	85.27% <100.00%> (?)
unittests-Instrumentation.EventCounters	76.36% <ø> (?)
unittests-Instrumentation.GrpcNetClient	78.78% <ø> (?)
unittests-Instrumentation.Http	81.79% <ø> (?)
unittests-Instrumentation.Owin	83.43% <ø> (?)
unittests-Instrumentation.Process	100.00% <ø> (?)
unittests-Instrumentation.Runtime	100.00% <ø> (?)
unittests-Instrumentation.SqlClient	90.90% <ø> (?)
unittests-Instrumentation.StackExchangeRedis	71.00% <ø> (?)
unittests-Instrumentation.Wcf	78.47% <ø> (?)
unittests-PersistentStorage	65.78% <ø> (?)
unittests-ResourceDetectors.Azure	81.53% <ø> (?)
unittests-ResourceDetectors.Host	54.11% <ø> (?)
unittests-ResourceDetectors.Process	100.00% <ø> (?)
unittests-ResourceDetectors.ProcessRuntime	76.08% <ø> (?)
unittests-Resources.Gcp	72.54% <ø> (?)
unittests-Solution	80.17% <ø> (?)

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

Files	Coverage Δ
...umentation.AspNetCore/AspNetCoreInstrumentation.cs	100.00% <100.00%> (ø)
...eInstrumentationTracerProviderBuilderExtensions.cs	100.00% <ø> (ø)
...ry.Instrumentation.AspNetCore/AspNetCoreMetrics.cs	100.00% <100.00%> (ø)
...spNetCore/AspNetCoreTraceInstrumentationOptions.cs	95.23% <100.00%> (ø)
...tation.AspNetCore/Implementation/HttpInListener.cs	77.02% <100.00%> (ø)
...AspNetCore/Implementation/HttpInMetricsListener.cs	89.74% <100.00%> (ø)

... and 276 files with indirect coverage changes

[vishweshbankwar]

Could we use Activity.Current!? IIRC, this is already checked at caller side.

[Kielek]

Fixed: 1aaea52