fix: make RunCommandLineApplicationAsync actually async

Author: natemcmaster

azure-pipelines.yml

@@ -36,8 +36,6 @@ stages:
       displayName: Invoke build.ps1
       env:
         KEYVAULT_CLIENT_SECRET: $(kv-access-token)
-    - powershell: .\docs\generate.ps1 -Verbose -NoBuild
-      displayName: Compile documentation
     - task: PublishTestResults@2
       displayName: Upload test results
       condition: always()

src/Hosting.CommandLine/Internal/CommandLineService.cs

@@ -44,12 +44,11 @@ public CommandLineService(ILogger<CommandLineService<T>> logger, CommandLineStat
         }
 
         /// <inheritdoc />
-        public Task<int> RunAsync(CancellationToken cancellationToken)
+        public async Task<int> RunAsync(CancellationToken cancellationToken)
         {
             _logger.LogDebug("Running");
-            // TODO support cancellation tokens. See #111
-            _state.ExitCode = _application.Execute(_state.Arguments);
-            return Task.FromResult(_state.ExitCode);
+            _state.ExitCode = await _application.ExecuteAsync(_state.Arguments, cancellationToken);
+            return _state.ExitCode;
         }
 
         public void Dispose()

src/Hosting.CommandLine/releasenotes.props

@@ -0,0 +1,14 @@
+<Project>
+  <PropertyGroup>
+    <PackageReleaseNotes Condition="'$(VersionPrefix)' == '2.4.0'">
+Changes:
+* Support C# 8.0 and nullable reference types
+* RunCommandLineApplicationAsync is actually async now, not just sync disguised as an async API
+* Better CTRL+C support
+    </PackageReleaseNotes>
+    <PackageReleaseNotes>$(PackageReleaseNotes)
+
+See more details here: https://github.com/natemcmaster/CommandLineUtils/blob/master/CHANGELOG.md#v$(VersionPrefix.Replace('.',''))
+    </PackageReleaseNotes>
+  </PropertyGroup>
+</Project>