When will pull request #777 be merged?

inversify / InversifyJS

A powerful and lightweight inversion of control container for JavaScript & Node.js apps powered by TypeScript.

http://inversify.io/

MIT License

11.3k stars 718 forks source link

When will pull request #777 be merged? #955

Open zveljkovic opened 6 years ago

zveljkovic commented 6 years ago

A lot of utility classes are meant to be in singleton scope i.e. DbHelper. When we want to test some parts of code we need to swap them out. If we use DI across whole project we need the singletons to work properly with snapshot() and restore() as it's useful in tests before() and after() global hooks.

Expected Behavior

To include a pull request in reasonable amount of time

Current Behavior

Pull request sits for 8 months. If something is missing please write so we can amend.

dcavanagh commented 6 years ago

@remojansen There was a conversation going on between you and @m3l7. can you please advise if this PR is ready?