Skip to content
This repository has been archived by the owner on Nov 3, 2018. It is now read-only.

What about renaming PuzzleApplicationBootstrapper in ApplicationBootstrapper? #28

Open
micdenny opened this issue Dec 28, 2015 · 3 comments · May be fixed by #32
Open

What about renaming PuzzleApplicationBootstrapper in ApplicationBootstrapper? #28

micdenny opened this issue Dec 28, 2015 · 3 comments · May be fixed by #32

Comments

@micdenny
Copy link
Member

or at least adding a shortcut if it's possibile, just because we would like to have the PuzzleApplicationBootstrapper our default easy peasy container, so it could be cool to have it named ApplicationBootstrapper and probably under Radical.Windows namespace.

I saw doing such things in new frameworks out there (like EF7), they use to add the entry point class/methods in the very first namespace where they belong to (eg.: Microsoft.Data.Entity), to avoid searching the right namespace to add, and it could also be great to have extension method to enable different containers, so to have something like:

using Radical.Windows;

var bootstrapper = new ApplicationBootstrapper<Presentation.MainView>();

when using just the internal container (puzzle), or this:

using Radical.Windows;

var bootstrapper = new ApplicationBootstrapper<Presentation.MainView>().UseCastleWindsor();

you see, the same namespace is imported Radical.Windows, and you have only to add the proper nuget package to have in the same namespace the extension method UseCastleWindsor(), so no need to find in other namespace.

how cool could it be? 😄

@mauroservienti
Copy link
Member

Very cool. I love it.
Basically we could split the Bootstrapper in at least 2 pieces:

  • Application logic bootstrapper
  • Container bootstrapper

The container one can be pluggable using the extension methods logic you proposed. I also think that the same approach should be used at the full fx level.

@mauroservienti
Copy link
Member

Depends on #23

@mauroservienti
Copy link
Member

Moving this back to the ready state since we need #23 before moving on with this

@mauroservienti mauroservienti removed their assignment Feb 26, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants