Remove TODO as FakeAppDomain is obsolete, and also automatic assembly loading (#5169)

Author: snakefoot

src/NLog/Config/ConfigurationItemFactory.cs

@@ -546,7 +546,6 @@ internal static IEnumerable<KeyValuePair<string, string>> GetAutoLoadingFileLoca
             if (!string.IsNullOrEmpty(entryAssemblyLocation) && !string.Equals(entryAssemblyLocation, nlogAssemblyLocation, StringComparison.OrdinalIgnoreCase))
                 yield return new KeyValuePair<string, string>(entryAssemblyLocation, nameof(entryAssemblyLocation));
 
-            // TODO Consider to prioritize AppDomain.PrivateBinPath
             var baseDirectory = PathHelpers.TrimDirectorySeparators(LogFactory.DefaultAppEnvironment.AppDomainBaseDirectory);
             InternalLogger.Debug("Auto loading based on AppDomain-BaseDirectory found location: {0}", baseDirectory);
             if (!string.IsNullOrEmpty(baseDirectory) && !string.Equals(baseDirectory, nlogAssemblyLocation, StringComparison.OrdinalIgnoreCase))

src/NLog/Internal/AppDomains/FakeAppDomain.cs

@@ -126,7 +126,7 @@ private static string GetFriendlyNameFromProcessName()
         /// <returns>A list of assemblies in this application domain.</returns>
         public IEnumerable<Assembly> GetAssemblies()
         {
-            return Internal.ArrayHelper.Empty<Assembly>();  // TODO NETSTANDARD1_6 has DependencyContext.RuntimeLibraries
+            return Internal.ArrayHelper.Empty<Assembly>();
         }
 
         /// <summary>