small fixes

.editorconfig

@@ -93,6 +93,7 @@ dotnet_naming_symbols.constant_fields.required_modifiers          = const
 dotnet_style_operator_placement_when_wrapping = beginning_of_line
 tab_width = 2
 end_of_line = crlf
+dotnet_diagnostic.CA2007.severity = suggestion
 ###############################
 # C# Coding Conventions       #
 ###############################
@@ -960,6 +961,7 @@ csharp_style_prefer_method_group_conversion = true:silent
 csharp_style_prefer_top_level_statements = true:silent
 csharp_style_expression_bodied_lambdas = true:silent
 csharp_style_expression_bodied_local_functions = false:silent
+csharp_style_prefer_primary_constructors = true:suggestion
 
 
 [*.vb]

azure-pipelines.yml

@@ -142,7 +142,7 @@ stages:
 
         - template: ../steps/CoverageResults.yml
           parameters:
-            reports: $(Build.SourcesDirectory)/artifacts/TestResults/**/coverage.cobertura.xml
+            reports: $(Build.SourcesDirectory)/**/coverage.cobertura.xml
             configuration: $(_BuildConfig)
             condition: and(succeeded(), eq(variables['_BuildConfig'], 'Debug'))
             assemblyfilters: '-xunit'

eng/azuredevops/pipelines/steps/CoverageResults.yml

@@ -20,6 +20,11 @@ steps:
     verbosity: 'Verbose'
     assemblyfilters: ${{parameters.assemblyfilters}}
 
+- publish: '$(Build.SourcesDirectory)/artifacts/TestResults'
+  displayName: 'Publish TestResults Artifacts'
+  artifact: TestResults_$(Agent.Os)_$(_BuildConfig)
+  condition: ${{parameters.condition}}
+
 - publish: '$(Build.SourcesDirectory)/artifacts/CoverageReport'
   displayName: 'Publish CoverageReport Artifact'
   artifact: CoverageResults_$(Agent.Os)_${{parameters.configuration}}

src/RazorPages/RazorPagesIntro/Pages/Error.cshtml.cs

@@ -12,7 +12,9 @@ public class ErrorModel : PageModel
 
         public bool ShowRequestId => !string.IsNullOrEmpty(RequestId);
 
+#pragma warning disable S4487 // Unread "private" fields should be removed
         private readonly ILogger<ErrorModel> _logger;
+#pragma warning restore S4487 // Unread "private" fields should be removed
 
         public ErrorModel(ILogger<ErrorModel> logger)
         {

src/RazorPages/RazorPagesIntro/Pages/Privacy.cshtml.cs

@@ -5,7 +5,9 @@ namespace RazorPagesIntro.Pages
 {
     public class PrivacyModel : PageModel
     {
+#pragma warning disable S4487 // Unread "private" fields should be removed
         private readonly ILogger<PrivacyModel> _logger;
+#pragma warning restore S4487 // Unread "private" fields should be removed
 
         public PrivacyModel(ILogger<PrivacyModel> logger)
         {
@@ -14,6 +16,7 @@ public PrivacyModel(ILogger<PrivacyModel> logger)
 
         public void OnGet()
         {
+            // to be populated
         }
     }
 }

src/RazorPages/RazorPagesTestSample/Pages/Partials.cshtml.cs

@@ -7,6 +7,7 @@ public class PartialsModel : PageModel
     {
         public void OnGet()
         {
+            // to be populated
         }
 
         public IActionResult OnGetPartial() => Partial("_Partial");

src/RazorPages/RazorPagesTestSample/Program.cs

@@ -8,7 +8,11 @@
 
 namespace RazorPagesTestSample
 {
+#pragma warning disable CA1052 // Static holder types should be Static or NotInheritable
+#pragma warning disable S1118
     public class Program
+#pragma warning restore S1118
+#pragma warning restore CA1052 // Static holder types should be Static or NotInheritable
     {
         public static void Main(string[] args)
         {