fix(app): new we don't know if host already built or not

sitkoru · Dec 6, 2023 · 0910679 · 0910679
1 parent f7b164a
commit 0910679
Showing 1 changed file with 0 additions and 7 deletions.
diff --git a/src/Sitko.Core.App/SitkoCoreApplicationBuilder.cs b/src/Sitko.Core.App/SitkoCoreApplicationBuilder.cs
@@ -117,18 +117,11 @@ public SitkoCoreApplicationBuilder AddModule<TModule, TModuleOptions>(
     public bool HasModule<TModule>() where TModule : IApplicationModule =>
         moduleRegistrations.Any(r => r.Type == typeof(TModule));
 
-    private bool CanAddModule() => moduleRegistrations.Count < 100500; // TODO: still needed?
-
     private void RegisterModule<TModule, TModuleOptions>(
         Action<IApplicationContext, TModuleOptions>? configureOptions = null,
         string? optionsKey = null)
         where TModule : IApplicationModule<TModuleOptions>, new() where TModuleOptions : BaseModuleOptions, new()
     {
-        if (!CanAddModule())
-        {
-            throw new InvalidOperationException("App host is already built. Can't add modules after it");
-        }
-
         var instance = new TModule();
         if (!instance.AllowMultiple && HasModule<TModule>())
         {