Enable Nulls on Contrib.Extensions.AWSXRay & Test Projects

[FernandoNunes]

Fixes #.

Changes

Enabled Nulls (enable) on Contrib.Extensions.AWSXRay & Test Projects

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: FernandoNunes / name: Fernando Nunes (db97178, 9d42863)

[Kielek]

@FernandoNunes, please sign CLA, without this we want be able to merge your code.

I have approved run, but I expect a lot of issues related to the nullability part.

[FernandoNunes]

Hi @Kielek , I didn't noticed the project had warnings, my bad. I've commited the changes required for nullability checks and all tests are passing (netcoreapp3.1 & net452).

Also, I've signed the CLA.

[Kielek]

There are some compilation issues. Could you please address it?

[FernandoNunes]

There are some compilation issues. Could you please address it?

Added missing #nullable enable
Sorry still getting the hang of it, I'll be more efficient next PRs.

[FernandoNunes]

Visual Studio suggests the fixes to PublicAPI but doesn't changes for all platform's PublicAPI.txts.
After all changes are applied to one, I've started reviewing them all by hand for all platforms.

[Kielek]

Visual Studio suggests the fixes to PublicAPI but doesn't changes for all platform's PublicAPI.txts. After all changes are applied to one, I've started reviewing them all by hand for all platforms.

Unfortunately, it is the only way I know how to fix it.

[github-actions]

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

[github-actions]

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

[srprash]

@FernandoNunes The changes look good to me. Can you resolve the conflicts on this PR? Thanks!

[codecov]

Codecov Report

Merging #920 (c96b0ec) into main (e6a714d) will decrease coverage by 0.02%.
The diff coverage is 90.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #920      +/-   ##
==========================================
- Coverage   69.46%   69.44%   -0.02%     
==========================================
  Files         202      202              
  Lines        7663     7668       +5     
==========================================
+ Hits         5323     5325       +2     
- Misses       2340     2343       +3     

Impacted Files	Coverage Δ
...y.Contrib.Extensions.AWSXRay/AWSXRayIdGenerator.cs	61.64% <ø> (ø)
...ources/Http/ServerCertificateValidationProvider.cs	60.00% <50.00%> (-1.30%)	⬇️
...ntrib.Extensions.AWSXRay/Resources/Http/Handler.cs	70.58% <66.66%> (-2.75%)	⬇️
...nsions.AWSXRay/Resources/AWSEKSResourceDetector.cs	57.14% <83.33%> (ø)
...nsions.AWSXRay/Resources/AWSEBSResourceDetector.cs	87.50% <100.00%> (ø)
...nsions.AWSXRay/Resources/AWSEC2ResourceDetector.cs	68.96% <100.00%> (ø)
...nsions.AWSXRay/Resources/AWSECSResourceDetector.cs	67.85% <100.00%> (ø)
...ons.AWSXRay/Resources/AWSLambdaResourceDetector.cs	84.21% <100.00%> (ø)
...ns.AWSXRay/Resources/Models/AWSEBSMetadataModel.cs	100.00% <100.00%> (ø)
...ay/Resources/Models/AWSEC2IdentityDocumentModel.cs	100.00% <100.00%> (ø)
... and 3 more

[FernandoNunes]

@FernandoNunes The changes look good to me. Can you resolve the conflicts on this PR? Thanks!

All checks have passed. After this one can you check other PRs I have open ? :)