Address code review remarks.

Author: AndriySvyryd

src/EFCore/Properties/CoreStrings.Designer.cs

@@ -1084,7 +1084,7 @@
     <value>The index with name {indexName} cannot be removed from the entity type '{entityType}' because no such index exists on that entity type.</value>
   </data>
   <data name="NativeAotNoCompiledModel" xml:space="preserve">
-    <value>Model building is not supported when publishing with NativeAOT. Use a compiled model. </value>
+    <value>Model building is not supported when publishing with NativeAOT. Use a compiled model.</value>
   </data>
   <data name="NavigationArray" xml:space="preserve">
     <value>The type of navigation '{1_entityType}.{0_navigation}' is '{foundType}' which is an array type. Collection navigations cannot be arrays.</value>

src/EFCore/Properties/CoreStrings.resx

@@ -15,7 +15,7 @@ Microsoft.Data.Sqlite.SqliteException
 Microsoft.Data.Sqlite.SqliteFactory
 Microsoft.Data.Sqlite.SqliteParameter
 Microsoft.Data.Sqlite.SqliteTransaction</Description>
-    <TargetFrameworks>$(DefaultNetCoreTargetFramework);netstandard2.0</TargetFrameworks>
+    <TargetFrameworks>$(DefaultNetCoreTargetFramework);netstandard2.0;net6.0</TargetFrameworks>
     <MinClientVersion>3.6</MinClientVersion>
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <CodeAnalysisRuleSet>Microsoft.Data.Sqlite.Core.ruleset</CodeAnalysisRuleSet>

src/Microsoft.Data.Sqlite.Core/Microsoft.Data.Sqlite.Core.csproj

@@ -315,7 +315,7 @@ public override string GetDataTypeName(int ordinal)
         /// </summary>
         /// <param name="ordinal">The zero-based column ordinal.</param>
         /// <returns>The data type of the column.</returns>
-#if NET6_0_OR_GREATER
+#if NET8_0_OR_GREATER
         [return: DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.PublicProperties | DynamicallyAccessedMemberTypes.PublicFields)]
 #endif
         public override Type GetFieldType(int ordinal)
@@ -609,11 +609,6 @@ public override int GetValues(object?[] values)
         /// </summary>
         /// <returns>A System.Data.DataTable that describes the column metadata.</returns>
         /// <seealso href="https://docs.microsoft.com/dotnet/standard/data/sqlite/metadata">Metadata</seealso>
-#if NET6_0_OR_GREATER
-        [UnconditionalSuppressMessage("Trimming", "IL2111:Method with parameters or return value with `DynamicallyAccessedMembersAttribute`"
-            + " is accessed via reflection. Trimmer can't guarantee availability of the requirements of the method.",
-            Justification = "This is about System.Type.TypeInitializer.get. Everything needed for DataColumn is preserved.")]
-#endif
         public override DataTable GetSchemaTable()
         {
             if (_closed)
@@ -633,8 +628,7 @@ public override DataTable GetSchemaTable()
             var ColumnSize = new DataColumn(SchemaTableColumn.ColumnSize, typeof(int));
             var NumericPrecision = new DataColumn(SchemaTableColumn.NumericPrecision, typeof(short));
             var NumericScale = new DataColumn(SchemaTableColumn.NumericScale, typeof(short));
-
-            var DataType = new DataColumn(SchemaTableColumn.DataType, typeof(Type));
+            var DataType = CreateDataTypeColumn();
             var DataTypeName = new DataColumn("DataTypeName", typeof(string));
 
             var IsLong = new DataColumn(SchemaTableColumn.IsLong, typeof(bool));
@@ -756,6 +750,18 @@ public override DataTable GetSchemaTable()
             }
 
             return schemaTable;
+
+#if NET6_0_OR_GREATER
+            [UnconditionalSuppressMessage("Trimming", "IL2111:Method with parameters or return value with `DynamicallyAccessedMembersAttribute`"
+                + " is accessed via reflection. Trimmer can't guarantee availability of the requirements of the method.",
+                Justification = "This is about System.Type.TypeInitializer.get. It is accessed via reflection"
+                + " as the type parameter in DataColumn is annotated with DynamicallyAccessedMemberTypes.PublicFields | DynamicallyAccessedMemberTypes.PublicProperties" +
+                " However, everything needed for DataColumn is preserved, there's test coverage for this code.")]
+#endif
+            static DataColumn CreateDataTypeColumn()
+            {
+                return new DataColumn(SchemaTableColumn.DataType, typeof(Type));
+            }
         }
     }
 }

src/Microsoft.Data.Sqlite.Core/SqliteDataReader.cs

@@ -21,7 +21,6 @@ Microsoft.Data.Sqlite.SqliteTransaction</Description>
     <IncludeSymbols>false</IncludeSymbols>
     <DisableImplicitFrameworkReferences>true</DisableImplicitFrameworkReferences>
     <PackageProjectUrl>https://docs.microsoft.com/dotnet/standard/data/sqlite/</PackageProjectUrl>
-    <IsAotCompatible>true</IsAotCompatible>
   </PropertyGroup>
 
   <ItemGroup>