Move Program.Main into WebApplication.Start.

[Tratcher]

#430 @davidfowl @DamianEdwards @muratg

This is a really strange class and it's going to confuse anyone that used Katana. It's especially strange that Start never returns. Can we rename Start to Main or Run?

[muratg]

Perhaps we should keep the Program class, and add the Start methods there?

[Tratcher]

public static void Main(string[] args) => Program.Start<Startup>(args);? Hmm, I think that looses some context because you don't see who's Program your calling. Nor does it address the name Start.

[davidfowl]

Call it run instead of start

[davidfowl]

Also optionally take in the startup class

[Tratcher]

An instance of the startup class rather than just the type? We don't actually support that anywhere in Hosting right now and it won't be trivial to add. We can work on that later.

[davidfowl]

@Tratcher no, WebApplication.Run<TStartup>()

[davidfowl]

Oh wait you have that 🔹

[davidfowl]

Hmmm hold on

[davidfowl]

We need to tweak this a bit. Use this logic:

https://github.com/aspnet/Configuration/blob/dev/src/Microsoft.Extensions.Configuration.FileExtensions/FileConfigurationExtensions.cs#L52-L57

To resolve the application base path

[Tratcher]

Why isn't that done inside of PlatformServices.Default.Application.ApplicationBasePath?

[davidfowl]