Remove the FactoryModule (replaced by Ninject.Extensions.Factory) and fix unit tests #12

Open
wants to merge 3 commits into
from

Projects

None yet

2 participants

@scott-xu

No description provided.

@scott-xu

This would be a breaking change as user need to refer Ninject.Extensions.Factory if they used/will use Func<>.
Where can I document this?

@khellang
Custodians of the Super-Duper-Happy-Path member

I'm not sure I'd classify this a a breaking change. It's just a new dependency and we don't have to change public APIs because of it. The question is; why is it there in the first place? Do we use Func<T> factories anywhere? @thecodejunkie, @grumpydev? If we do, then this gets my 👍

@scott-xu scott-xu Ninject.Extensions.Factory
Add Ninject.Extensions.Factory dependency
35b320b
@scott-xu

Hi @thecodejunkie @grumpydev, what do you think about this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment