ProCoSys / Bifrost

Application Development Framework promoting good development practices such as CQRS and MVVM (see readme)
http://bifrost.dolittle.com
Other
1 stars 1 forks source link

Configure.DiscoverAndConfigure is not SOLID #30

Closed bnordli closed 8 years ago

bnordli commented 8 years ago

Configure.DiscoverAndConfigure blatantly violates the Single Responsibility Principle by newing up 12 instances of different types and doing everything from setting up default values (where null is supplied) to discovering assemblies, to finally creating a container.

Bootstrapping the container should be refactored out to a separate system.

andersnygaard commented 8 years ago

Fixed by pull request #37