Code cleanup and typo fix (#7393)

* Drop IsExternalInit.cs

* Fixed a wrong cref namespace in comment

* Mark IWorkflowHost obsoleted too

* Remove Source Link package reference

Starting with .NET 8, Source Link for GitHub is included in the .NET SDK and enabled by default.

https://github.com/dotnet/sourcelink?tab=readme-ov-file#using-source-link-in-net-projects

Directory.Packages.props

@@ -131,7 +131,6 @@
         <PackageVersion Include="Microsoft.Data.SqlClient" Version="6.1.3"/>
         <PackageVersion Include="Microsoft.Identity.Client" Version="4.79.2"/>
         <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="18.0.1"/>
-        <PackageVersion Include="Microsoft.SourceLink.GitHub" Version="8.0.0" PrivateAssets="All"/>
         <PackageVersion Include="Moq" Version="4.20.72"/>
         <PackageVersion Include="MySql.Data" Version="9.5.0"/>
         <PackageVersion Include="Newtonsoft.Json" Version="13.0.4"/>

src/Directory.Build.props

@@ -14,7 +14,6 @@
     <PackageReference Include="ConfigureAwait.Fody" PrivateAssets="All" />
     <PackageReference Include="Fody" PrivateAssets="All" />
     <PackageReference Include="JetBrains.Annotations" PrivateAssets="All" />
-    <PackageReference Include="Microsoft.SourceLink.GitHub" PrivateAssets="All" />
   </ItemGroup>
 
 </Project>

src/apps/Directory.Build.props

@@ -12,6 +12,5 @@
         <PackageReference Include="ConfigureAwait.Fody" PrivateAssets="All"/>
         <PackageReference Include="Fody" PrivateAssets="All"/>
         <PackageReference Include="JetBrains.Annotations" PrivateAssets="All"/>
-        <PackageReference Include="Microsoft.SourceLink.GitHub" PrivateAssets="All"/>
     </ItemGroup>
 </Project>

src/clients/Elsa.Api.Client/IsExternalInit.cs

@@ -1,7 +0,0 @@
-using System.ComponentModel;
-
-// ReSharper disable once CheckNamespace
-namespace System.Runtime.CompilerServices;
-
-[EditorBrowsable(EditorBrowsableState.Never)]
-internal sealed class IsExternalInit;

src/modules/Elsa.Workflows.Core/Activities/Flowchart/Models/FlowchartExecutionMode.cs

@@ -6,7 +6,7 @@ namespace Elsa.Workflows.Activities.Flowchart.Models;
 public enum FlowchartExecutionMode
 {
     /// <summary>
-    /// Use the default mode as specified by <see cref="Elsa.Workflows.Activities.Flowchart.Flowchart.UseTokenFlow"/>.
+    /// Use the default mode as specified by <see cref="Activities.Flowchart.UseTokenFlow"/>.
     /// </summary>
     Default = 0,
 

src/modules/Elsa.Workflows.Core/IsExternalInit.cs

@@ -1,17 +0,0 @@
-// // Licensed to the .NET Foundation under one or more agreements.
-// // The .NET Foundation licenses this file to you under the MIT license.
-//
-// using System.ComponentModel;
-//
-// // ReSharper disable once CheckNamespace
-// namespace System.Runtime.CompilerServices
-// {
-//     /// <summary>
-//     /// Reserved to be used by the compiler for tracking metadata.
-//     /// This class should not be used by developers in source code.
-//     /// </summary>
-//     [EditorBrowsable(EditorBrowsableState.Never)]
-//     internal static class IsExternalInit
-//     {
-//     }
-// }

src/modules/Elsa.Workflows.Runtime/Contracts/IWorkflowHost.cs

@@ -8,6 +8,7 @@ namespace Elsa.Workflows.Runtime;
 /// <summary>
 /// Represents a single workflow instance that can be executed and takes care of publishing various lifecycle events.
 /// </summary>
+[Obsolete("Use IWorkflowRuntime and IWorkflowClient services instead.")]
 public interface IWorkflowHost
 {
     /// <summary>

src/modules/Elsa.Workflows.Runtime/IsExternalInit.cs

@@ -1,16 +0,0 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
-
-using System.ComponentModel;
-
-// ReSharper disable once CheckNamespace
-namespace System.Runtime.CompilerServices;
-
-/// <summary>
-/// Reserved to be used by the compiler for tracking metadata.
-/// This class should not be used by developers in source code.
-/// </summary>
-[EditorBrowsable(EditorBrowsableState.Never)]
-internal static class IsExternalInit
-{
-}