Problem
Both DBTools.csproj and DBToolsUnitTest.csproj set <Nullable>disable</Nullable>, missing compile-time null safety that is standard for modern .NET libraries.
Proposed solution
- Enable
<Nullable>enable</Nullable> in DBTools.csproj
- Fix nullability warnings in public API surface first (
Core/, Abstractions/, Configuration/)
- Enable in test project after library compiles cleanly
- Add
#nullable enable incrementally per file if a big-bang migration is too large
Priority areas:
DbConfiguration (config values from JSON)
- LINQ expression translation return types
GenericObject property arrays
Acceptance criteria
Notes
Can be done incrementally with warningsAsErrors optional in a follow-up.
Problem
Both
DBTools.csprojandDBToolsUnitTest.csprojset<Nullable>disable</Nullable>, missing compile-time null safety that is standard for modern .NET libraries.Proposed solution
<Nullable>enable</Nullable>inDBTools.csprojCore/,Abstractions/,Configuration/)#nullable enableincrementally per file if a big-bang migration is too largePriority areas:
DbConfiguration(config values from JSON)GenericObjectproperty arraysAcceptance criteria
Nullableenabled in library project?,!) are accurateNotes
Can be done incrementally with
warningsAsErrorsoptional in a follow-up.