Consolidated changes from #1334

Author: bart-vmware

.github/ISSUE_TEMPLATE/bug_report.md

@@ -8,9 +8,9 @@ assignees: ''
 ---
 
 ## Describe the bug
-A clear and concise description of the bug.  
+A clear and concise description of the bug.
 
-## Steps to reproduce 
+## Steps to reproduce
 Steps to reproduce the behavior:
 1.
 2.
@@ -20,9 +20,9 @@ A clear and concise description of what you expected to happen.
 
 ## Environment (please complete the following information):
  - Steeltoe Version [e.g. 2.3.0]
- - Platform: [e.g. Cloud Foundry, Azure, etc.) 
+ - Platform: [e.g. Cloud Foundry, Azure, etc.)
  - OS: [e.g. Windows, Linux, Mac OSX]
- - .NET Version [e.g. .NET Core 2.1.0, .NET Framework 4.7.1, etc.] 
+ - .NET Version [e.g. .NET Core 2.1.0, .NET Framework 4.7.1, etc.]
  - Any other library versions to note
 
 ## Screenshots

.github/ISSUE_TEMPLATE/question.md

@@ -11,7 +11,7 @@ assignees: ''
 _Enter your question or feedback here_
 
 ## Environment (please complete the following information):
- - .NET Version [e.g. .NET Core 3.1.4, .NET Framework 4.7.1, etc.] 
+ - .NET Version [e.g. .NET Core 3.1.4, .NET Framework 4.7.1, etc.]
  - Steeltoe Version [e.g. 3.0.1]
 
 ## Additional context or links

build/pr-code-cleanup.yml

@@ -82,7 +82,7 @@ jobs:
     displayName: Push changes
     condition: and(succeeded(), eq(variables['hasChangesToCommit'], 'True'))
     env:
-      SYSTEM_ACCESSTOKEN: $(System.AccessToken)  
+      SYSTEM_ACCESSTOKEN: $(System.AccessToken)
     inputs:
       targetType: 'inline'
       script: |
@@ -112,7 +112,7 @@ jobs:
           $statusMessage = "<samp>Code cleanup was canceled, no changes were pushed.</samp>"
         }
         elseif ($env:AGENT_JOBSTATUS -eq "Failed") {
-          $url = "$(System.TeamFoundationCollectionUri)$(System.TeamProject)/_build/results?buildId=$(Build.BuildId)" 
+          $url = "$(System.TeamFoundationCollectionUri)$(System.TeamProject)/_build/results?buildId=$(Build.BuildId)"
           $statusMessage = "<samp>Code cleanup failed to reformat and push changes.</samp><p>View details [here]($url).</p>"
         }
         else {

build/sign-packages.ps1

@@ -16,7 +16,7 @@ $appSettings = Join-Path $currentDirectory 'appsettings.json'
 $nupkgs = Get-ChildItem $env:ArtifactDirectory/Steeltoe*.*nupkg -recurse | Select-Object -ExpandProperty FullName
 
 foreach ($nupkg in $nupkgs) {
-	./SignClient 'sign' -c $appSettings -i $nupkg -r $env:SignClientUser -s $env:SignClientSecret -n 'Steeltoe' -d 'Steeltoe' -u 'https://github.com/SteeltoeOSS' 
+	./SignClient 'sign' -c $appSettings -i $nupkg -r $env:SignClientUser -s $env:SignClientSecret -n 'Steeltoe' -d 'Steeltoe' -u 'https://github.com/SteeltoeOSS'
 }
 
 Write-Host "Sign-packages completed."

build/templates/consolidate-coverage.yaml

@@ -28,4 +28,4 @@ jobs:
     displayName: Publish code coverage to Azure DevOps
     inputs:
       codeCoverageTool: Cobertura
-      summaryFileLocation: $(Build.ArtifactStagingDirectory)/CodeCoverage/Cobertura.xml
+      summaryFileLocation: $(Build.ArtifactStagingDirectory)/CodeCoverage/Cobertura.xml

build/verify-code-style.yml

@@ -51,10 +51,10 @@ jobs:
           # We are in detached head (the merge result), so there's no need to account for an out-of-date PR.
           $baseCommitHash = git rev-parse origin/$env:SYSTEM_PULLREQUEST_TARGETBRANCH
           if ($LastExitCode -ne 0) { throw "Command 'git rev-parse (1)' failed with exit code $LastExitCode." }
-  
+
           $headCommitHash = git rev-parse HEAD
           if ($LastExitCode -ne 0) { throw "Command 'git rev-parse (2)' failed with exit code $LastExitCode." }
-  
+
           Write-Output "Running code cleanup on commit range $baseCommitHash..$headCommitHash in pull request."
           dotnet regitlint -s src/Steeltoe.All.sln --print-command --skip-tool-check --max-runs=5 --jb --dotnetcoresdk=$(dotnet --version) --jb-profile="Steeltoe Full Cleanup" --jb --properties:Configuration=Release --jb --properties:NuGetAudit=false --jb --verbosity=WARN --fail-on-diff --print-diff -f commits -a $headCommitHash -b $baseCommitHash
         }

coverlet.runsettings

@@ -4,7 +4,7 @@
     <DataCollectors>
       <DataCollector friendlyName="XPlat Code Coverage">
         <Configuration>
-          <Format>opencover</Format>          
+          <Format>opencover</Format>
           <Exclude>[*.Test*]*,[*]Microsoft.Diagnostics*</Exclude>
           <Include>[Steeltoe.*]*</Include>
           <ExcludeByAttribute>Obsolete,GeneratedCodeAttribute,CompilerGeneratedAttribute</ExcludeByAttribute>
@@ -15,4 +15,4 @@
       </DataCollector>
     </DataCollectors>
   </DataCollectionRunSettings>
-</RunSettings>
+</RunSettings>

shared-package.props

@@ -46,24 +46,11 @@
 
   <ItemGroup>
     <PackageReference Include="Microsoft.SourceLink.GitHub" Version="$(SourceLinkGitHubVersion)" PrivateAssets="All" />
+    <PackageReference Include="Microsoft.CodeAnalysis.PublicApiAnalyzers" Version="$(PublicApiAnalyzersVersion)" PrivateAssets="All" />
   </ItemGroup>
 
   <Target Name="ValidatePackage" AfterTargets="BeforeBuild">
     <Error Text="The 'Description' property must be set for packable project '$(MSBuildProjectName)'." Condition="'$(Description)' == ''" />
     <Error Text="The 'PackageTags' property must be set for packable project '$(MSBuildProjectName)'." Condition="'$(PackageTags)' == ''" />
   </Target>
-
-  <ItemGroup Condition="!$(MSBuildProjectName.StartsWith('Steeltoe.Common'))">
-    <!--
-      Remove the condition above after public API review of Steeltoe.Common.
-    -->
-    <PackageReference Include="Microsoft.CodeAnalysis.PublicApiAnalyzers" Version="$(PublicApiAnalyzersVersion)" PrivateAssets="All" />
-  </ItemGroup>
-
-  <PropertyGroup Condition="$(MSBuildProjectName.StartsWith('Steeltoe.Common'))">
-    <!--
-      Remove the condition above after public API review of Steeltoe.Common.
-    -->
-    <NoWarn>$(NoWarn);SA1401;S1168;S2360;S3900;S3956;S4004;S4023</NoWarn>
-  </PropertyGroup>
 </Project>

shared.props

@@ -1,6 +1,7 @@
 <Project>
   <PropertyGroup>
     <LangVersion>latest</LangVersion>
+    <Nullable>enable</Nullable>
     <ImplicitUsings>enable</ImplicitUsings>
     <EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild>
     <GenerateAssemblyCompanyAttribute>false</GenerateAssemblyCompanyAttribute>

src/Bootstrap/src/AutoConfiguration/BootstrapScanner.cs

@@ -47,9 +47,10 @@ internal sealed class BootstrapScanner
 
     public BootstrapScanner(HostBuilderWrapper wrapper, IReadOnlySet<string> assemblyNamesToExclude, ILoggerFactory loggerFactory)
     {
-        ArgumentGuard.NotNull(wrapper);
-        ArgumentGuard.NotNull(assemblyNamesToExclude);
-        ArgumentGuard.NotNull(loggerFactory);
+        ArgumentNullException.ThrowIfNull(wrapper);
+        ArgumentNullException.ThrowIfNull(assemblyNamesToExclude);
+        ArgumentGuard.ElementsNotNullOrWhiteSpace(assemblyNamesToExclude);
+        ArgumentNullException.ThrowIfNull(loggerFactory);
 
         _wrapper = wrapper;
         _loader = new AssemblyLoader(assemblyNamesToExclude);
@@ -61,7 +62,7 @@ public void ConfigureSteeltoe()
     {
         if (_loggerFactory is IBootstrapLoggerFactory)
         {
-            BootstrapLoggerHostedService.Register(_loggerFactory, _wrapper);
+            BootstrapLoggerHostedService.Register(_wrapper);
         }
 
         if (!WireIfLoaded(WireConfigServer, SteeltoeAssemblyNames.ConfigurationConfigServer))