Enable nullable reference types

[0xced]

Annotate all potentially nullable reference types accordingly

[martincostello]

This looks great - thanks @0xced!

[codecov]

Codecov Report

Merging #222 (d439447) into main (a3cd439) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #222   +/-   ##
=======================================
  Coverage   99.12%   99.12%           
=======================================
  Files           8        8           
  Lines         342      342           
=======================================
  Hits          339      339           
  Misses          3        3           

Flag	Coverage Δ
linux	99.12% <100.00%> (+0.87%)	⬆️
macos	99.12% <100.00%> (ø)
windows	99.12% <100.00%> (ø)

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

Impacted Files	Coverage Δ
src/Logging.XUnit/XUnitLoggerExtensions.cs	100.00% <ø> (ø)
...c/Logging.XUnit/AmbientTestOutputHelperAccessor.cs	100.00% <100.00%> (ø)
src/Logging.XUnit/TestOutputHelperAccessor.cs	100.00% <100.00%> (ø)
src/Logging.XUnit/XUnitLogScope.cs	100.00% <100.00%> (ø)
src/Logging.XUnit/XUnitLogger.cs	98.05% <100.00%> (ø)
src/Logging.XUnit/XUnitLoggerOptions.cs	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a3cd439...d439447. Read the comment docs.

[martincostello]

@0xced Would you mind trying out the prerelease build from MyGet with your change in your project(s) and check you're happy with the changes? If so, I'll push 0.1.2 to NuGet.org.

https://www.myget.org/feed/martincostello/package/nuget/MartinCostello.Logging.XUnit/0.1.2-beta0206

[0xced]

Actually, I'm not using this package, yet! I was planing another pull requests (support for IMessageSink in additino to ITestOutputHelper) and I thought it was safer to enable null reference types first. 😉

[martincostello]

Ah nice. In that case (unless it's going to be a very long time), I'll wait for that and the changes can go together 🙂