Skip to content

Commit 27198ac

Browse files
committed
Mark PopulateAppSettings as obsolete with a TODO for future reevaluation of its necessity and integration.
1 parent e4f3288 commit 27198ac

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/ServiceControl.Configuration/ExeConfiguration.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
namespace ServiceControl.Configuration
22
{
3+
using System;
34
using System.Configuration;
45
using System.IO;
56
using System.Linq;
@@ -10,6 +11,7 @@ public static class ExeConfiguration
1011
{
1112
// ConfigurationManager on .NET is looking for {assembly}.dll.config files, but all previous versions of ServiceControl will have {assembly}.exe.config instead.
1213
// This code reads in the exe.config files and adds all the values into the ConfigurationManager's collections.
14+
[Obsolete("TODO: Check if this is still needed if all settings or can be integrared in the LegacyAppSettingsConfigurationProvider or maybe even a new provider")]
1315
public static void PopulateAppSettings(Assembly assembly)
1416
{
1517
var location = Path.GetDirectoryName(assembly.Location);

0 commit comments

Comments
 (0)